Skip to content

@W-20048799 - Identification of action keyword during assessment#496

Merged
sf-pallavi-das merged 3 commits into
salesforcecli:developfrom
shaurabh-tiwari-git:u/shaurabh-tiwari/flexcardactionkeywordfix
May 29, 2026
Merged

@W-20048799 - Identification of action keyword during assessment#496
sf-pallavi-das merged 3 commits into
salesforcecli:developfrom
shaurabh-tiwari-git:u/shaurabh-tiwari/flexcardactionkeywordfix

Conversation

@shaurabh-tiwari-git
Copy link
Copy Markdown

What does this PR do?

What issues does this PR fix or reference?

@shaurabh-tiwari-git shaurabh-tiwari-git requested a review from a team as a code owner May 25, 2026 18:02
Comment thread src/migration/flexcard.ts
// "action" is a reserved keyword in the core runtime; flag any Custom LWC
// that defines it as a property so the user removes/renames it pre-migration.
if (Object.prototype.hasOwnProperty.call(component.property, 'action')) {
const lwcLabel = component.property.customlwcname || component.elementLabel || Constants.CustomLwc;
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

@shaurabh-tiwari-git As discussed over call, connect with Swati CX and add all the reserve keywords check with Flexcard custom attributes

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I've tried with other reserved keywords but they are not causing any issue in our case. They might be reserved for other places in the Flexcard but for our case only action keyword is failing in core runtime. Also, the PR by Adam in core for this issue (https://gitcore.soma.salesforce.com/core-2206/core-264-public/pull/2750) only addresses the "action" keyword.

Comment thread messages/assess.json Outdated
"cardNameChangeMessage": "The Flexcard name will be changed from %s to %s to adhere to the API naming standards.",
"authordNameChangeMessage": "The Flexcard author name will be changed from %s to %s to adhere to the naming rules",
"cardLWCNameChangeMessage": "The Flexcard generated LWC name will be changed from %s to %s to align with Flexcard name change.",
"customLwcReservedActionKey": "Custom LWC \"%s\" defines a reserved property key \"action\" which is not supported by the core runtime. Manual intervention required to remove or rename this key before migration.",
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Message is not correct - remove is not an option which will impact the functionality so it should be renamed not using reserve keywords which will be shared as documentation link

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Addressed

Comment thread src/utils/constants/documentRegistry.ts Outdated
'https://help.salesforce.com/s/articleView?id=xcloud.os_migrate_oma_prereq.htm&type=5',
corruptedParentChildLevel: 'https://help.salesforce.com/s/articleView?id=xcloud.os_version_omniscripts.htm&type=5',
customLwcReservedActionKey:
'https://help.salesforce.com/s/articleView?id=xcloud.os_embed_a_custom_lwc_inside_a_flexcard.htm&type=5',
Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

replace the link with CX created link which has clear steps on what should be done with this manual step

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

Addressed

@sf-pallavi-das sf-pallavi-das merged commit abe4b58 into salesforcecli:develop May 29, 2026
1 check passed
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.

4 participants