Skip to content

Feature/cmg 866#1029

Merged
umeshmore45 merged 5 commits intodevfrom
feature/cmg-866
Apr 7, 2026
Merged

Feature/cmg 866#1029
umeshmore45 merged 5 commits intodevfrom
feature/cmg-866

Conversation

@AishDani
Copy link
Copy Markdown
Contributor

@AishDani AishDani commented Apr 7, 2026

No description provided.

@AishDani AishDani requested a review from a team as a code owner April 7, 2026 06:52
@umeshmore45 umeshmore45 requested a review from Copilot April 7, 2026 07:11
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the Contentful entry transformation pipeline to better handle locale normalization/merging and attempts to ensure non-localized fields are present across locale branches before writing entry chunks.

Changes:

  • Added locale-merge logic so multiple Contentful locale branches that map to the same destination locale are combined before writing files.
  • Added logic intended to copy non-localized Contentful fields into all locale branches for an entry.
  • Adjusted UI field-type metadata for the taxonomy mapping option.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated 2 comments.

File Description
ui/src/components/ContentMapper/index.tsx Alters the taxonomy field-type configuration used by mapping/validation logic.
api/src/services/contentful.service.ts Adds helpers and logic to merge locales and to propagate non-localized fields across locales during entry transformation.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@umeshmore45 umeshmore45 merged commit dd1fcd5 into dev Apr 7, 2026
9 checks passed
@umeshmore45 umeshmore45 deleted the feature/cmg-866 branch April 7, 2026 11:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants