Skip to content

wire api fix#5294

Merged
jhefferman-sfdc merged 6 commits intomasterfrom
jhefferman/wire-api-fix
Mar 19, 2025
Merged

wire api fix#5294
jhefferman-sfdc merged 6 commits intomasterfrom
jhefferman/wire-api-fix

Conversation

@jhefferman-sfdc
Copy link
Contributor

@jhefferman-sfdc jhefferman-sfdc commented Mar 19, 2025

Details

SSRv2 wires were being invoked before component properties. If a property is both @api and @wire mutated, the api mutation takes precedence and the wire update is lost. This PR ensures wire invocation is last for V1/CSR parity.

This impacted a few scenarios, namely W-17953862.

Does this pull request introduce a breaking change?

  • 😮‍💨 No, it does not introduce a breaking change.

Does this pull request introduce an observable change?

  • 🤞 No, it does not introduce an observable change.

GUS work item

W-17953862

@jhefferman-sfdc jhefferman-sfdc requested a review from a team as a code owner March 19, 2025 01:34
@jhefferman-sfdc jhefferman-sfdc changed the base branch from jhefferman/wire-sequence-test to master March 19, 2025 13:46
@jhefferman-sfdc jhefferman-sfdc requested a review from wjhsf March 19, 2025 14:40
instance.connectedCallback();
__mutationTracker.disable(instance);
}
${/*connect wire*/ is.statement}
Copy link
Contributor

Choose a reason for hiding this comment

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

🎉🎉🎉

@jhefferman-sfdc jhefferman-sfdc merged commit 0a6c191 into master Mar 19, 2025
11 checks passed
@jhefferman-sfdc jhefferman-sfdc deleted the jhefferman/wire-api-fix branch March 19, 2025 16:22
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