Skip to content

Conversation

@sayalijoshi27
Copy link
Contributor

This PR contains :
CMG-771 - Existing stack -> Existing content type is selected -> when 2 matches found for a field and from that 1 is selected , cross icon is missing in text box

@sayalijoshi27 sayalijoshi27 requested a review from a team as a code owner January 22, 2026 06:18
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 pull request addresses a bug (CMG-771) where the cross icon (clear button) was missing from the text box when selecting a field from multiple matches, and includes package upgrades for various @contentstack CLI packages.

Changes:

  • Fixed the isClearable logic to show the clear button based on whether a field has a label rather than type matching
  • Simplified content type change handling to clear all field mappings instead of attempting to preserve them
  • Updated @contentstack CLI packages from version 1.5x.x to 1.55.0-1.17.0 range and related dependencies

Reviewed changes

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

File Description
ui/src/components/ContentMapper/index.tsx Fixed isClearable condition for the clear icon, added guard condition to prevent updates during content type changes, and simplified field mapping logic when content types are switched
api/package-lock.json Updated @contentstack CLI package versions, improved dependency declarations for @contentstack/cli-variants, and included various transitive dependency updates (ws, emoji-regex, iconv-lite, TypeScript, rollup, etc.)
Files not reviewed (1)
  • api/package-lock.json: Language not supported

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

@umeshmore45 umeshmore45 merged commit adf9328 into dev Jan 22, 2026
7 checks passed
@umeshmore45 umeshmore45 deleted the hotfix/package-upgrade branch January 22, 2026 09:01
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