Skip to content

Comments

Feature/wordpress#974

Merged
sayalijoshi27 merged 3 commits intodevfrom
feature/wordpress
Feb 23, 2026
Merged

Feature/wordpress#974
sayalijoshi27 merged 3 commits intodevfrom
feature/wordpress

Conversation

@AishDani
Copy link
Contributor

No description provided.

@AishDani AishDani requested a review from a team as a code owner February 23, 2026 12:58
@sayalijoshi27 sayalijoshi27 merged commit 01eb3de into dev Feb 23, 2026
9 checks passed
@sayalijoshi27 sayalijoshi27 deleted the feature/wordpress branch February 23, 2026 13:03
@AishDani AishDani restored the feature/wordpress branch February 23, 2026 13:03
Copy link
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 adds WordPress migration functionality with improvements to handle single-item/array data from WordPress XML exports and refines field mapping logic for nested modular block structures.

Changes:

  • Normalize WordPress XML category and term data to arrays to handle cases where XML parsing returns single objects instead of arrays
  • Add early return logic to prevent incorrect option display for deeply nested modular block fields
  • Add conditional filtering to exclude group fields from block field options at inappropriate nesting levels
  • Remove unused variables that were duplicating depth calculations

Reviewed changes

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

File Description
upload-api/migration-wordpress/libs/contentTypes.ts Normalizes rawCategories and rawTerms to arrays to handle WordPress XML data that may be single objects or arrays
ui/src/components/ContentMapper/index.tsx Improves field option filtering for modular blocks by adding depth-based guards and removing duplicate variables

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

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.

3 participants