Skip to content

Conversation

@yashin4112
Copy link
Contributor

This pr includes bugfix for the advance field reset issue.
Ticket Ref: https://contentstack.atlassian.net/browse/CMG-752

…ance field in the updated fields of handleResetContentType func
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 fixes a bug related to advance field reset (CMG-752) by preserving initial values when resetting fields and removing an unnecessary API call.

Key Changes:

  • Added initial property to the Advanced interface to store backup values for field reset functionality
  • Modified the field reset logic to restore advanced field values from their initial state
  • Removed a redundant fetchFields API call that was causing issues during content type reset

Reviewed Changes

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

File Description
ui/src/components/ContentMapper/contentMapper.interface.ts Added initial property to Advanced interface to support storing backup values for field reset
ui/src/components/ContentMapper/index.tsx Implemented field reset logic using initial values and removed redundant fetchFields call

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

@sayalijoshi27 sayalijoshi27 merged commit 49880db into dev Nov 13, 2025
7 checks passed
@sayalijoshi27 sayalijoshi27 deleted the bugfix/aem-752 branch November 13, 2025 11: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