Skip to content

feat: contentful integration #71

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom
Draft
wants to merge 10 commits into
base: main
Choose a base branch
from
Draft

feat: contentful integration #71

wants to merge 10 commits into from

Conversation

@tomaszpacior
Copy link
Collaborator

@tomaszpacior tomaszpacior commented Apr 3, 2025

Contentful integration implemented.

github-advanced-security[bot]
github-advanced-security bot found potential problems Apr 3, 2025
View reviewed changes
marcinkrasowski
marcinkrasowski reviewed Apr 4, 2025
View reviewed changes
packages/integrations/contentful-cms/public/images/empty.jpg Outdated
Copy link
Collaborator

@marcinkrasowski marcinkrasowski Apr 4, 2025

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I think these images can be removed

marcinkrasowski
marcinkrasowski reviewed Apr 4, 2025
View reviewed changes
packages/integrations/contentful-cms/src/modules/cms/cms.service.ts Outdated
include: 5,
});

const configurableTexts = this.cms.findEntries<IDataConfigurableTextsFields>({
Copy link
Collaborator

@marcinkrasowski marcinkrasowski Apr 4, 2025 •
edited
Loading

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

let's try to eliminate this request for every block, configurable texts could be included in CF on every page or block

@tomaszpacior tomaszpacior marked this pull request as draft April 4, 2025 12:20
tomaszpacior and others added 3 commits April 7, 2025 12:43
@tomaszpacior
feat: implement page mapper
f952846
@marcinkrasowski
Merge remote-tracking branch 'origin/main' into feature/contentful-in…
e5c9076 
…tegration

# Conflicts:
#	apps/api-harmonization/.env.local
#	apps/api-harmonization/turbo.json
#	package-lock.json
@marcinkrasowski
feat(integrations.contentful-cms): added mising mocks and fixed incor…
aad2753 
…rect mapping
@vercel vercel bot temporarily deployed to Preview – o2s-docs October 21, 2025 12:00 Inactive
@vercel
Copy link

vercel bot commented Oct 21, 2025 •
edited
Loading

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Preview Comments Updated (UTC)
o2s-docs Skipped Skipped Oct 27, 2025 11:59am

github-advanced-security[bot]
github-advanced-security bot found potential problems Oct 21, 2025
View reviewed changes
packages/integrations/contentful-cms/src/modules/cms/mappers/cms.page.mapper.ts
case slug.match(/\/uslugi\/.+/)?.[0]:
return {
...PAGE_SERVICE_DETAILS_PL,
slug: `/uslugi/${slug.match(/(.+)\/(.+)/)?.[2]}`,

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data High

This
regular expression
Loading
that depends on
library input
Loading
may run slow on strings with many repetitions of 'a'.
packages/integrations/contentful-cms/src/modules/cms/mappers/cms.page.mapper.ts
case slug.match(/\/orders\/.+/)?.[0]:
return {
...PAGE_ORDER_DETAILS_EN,
slug: `/orders/${slug.match(/(.+)\/(.+)/)?.[2]}`,

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data High

This
regular expression
Loading
that depends on
library input
Loading
may run slow on strings with many repetitions of 'a'.
packages/integrations/contentful-cms/src/modules/cms/mappers/cms.page.mapper.ts
case slug.match(/\/bestellungen\/.+/)?.[0]:
return {
...PAGE_ORDER_DETAILS_DE,
slug: `/bestellungen/${slug.match(/(.+)\/(.+)/)?.[2]}`,

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data High

This
regular expression
Loading
that depends on
library input
Loading
may run slow on strings with many repetitions of 'a'.
packages/integrations/contentful-cms/src/modules/cms/mappers/cms.page.mapper.ts
case slug.match(/\/zamowienia\/.+/)?.[0]:
return {
...PAGE_ORDER_DETAILS_PL,
slug: `/zamowienia/${slug.match(/(.+)\/(.+)/)?.[2]}`,

Check failure

Code scanning / CodeQL

Polynomial regular expression used on uncontrolled data High

This
regular expression
Loading
that depends on
library input
Loading
may run slow on strings with many repetitions of 'a'.
@vercel vercel bot temporarily deployed to Preview – o2s-docs October 27, 2025 11:59 Inactive
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@marcinkrasowski marcinkrasowski marcinkrasowski left review comments

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@tomaszpacior @marcinkrasowski