Skip to content

fix(event-form): stabilize page metadata (PPN) acknowledgment hydration#156

Merged
qiyundai merged 1 commit into
devfrom
fix/page-metadata-ppn-ack-hydration
May 20, 2026
Merged

fix(event-form): stabilize page metadata (PPN) acknowledgment hydration#156
qiyundai merged 1 commit into
devfrom
fix/page-metadata-ppn-ack-hydration

Conversation

@qiyundai
Copy link
Copy Markdown
Collaborator

@qiyundai qiyundai commented May 18, 2026

Resolves: https://jira.corp.adobe.com/browse/MWPW-193969

Summary

  • Apply publishing profile metadata and metadataFieldAcknowledged in a single form update when the metadata catalogue is already loaded; otherwise set pendingProfileAckBackfill and flush acks when the catalogue arrives.
  • Replace the profileHydrationTick effect with explicit pending backfill and merge previous ack state when syncing catalogue keys.
  • Reset publishingProfileRef and pending backfill when eventId changes so switching events always reloads the profile.
  • Normalize ESP responses with publishingProfileFromAssociation so profileId from the association wrapper is kept when the nested profile omits it.
  • Validation treats a field as satisfied when acknowledged or when metadata already holds a value (server-backed product selection).

Test plan

  • Webinar edit, profile with empty / missing PPN: change another step-1 field, save without opening the PPN picker — should succeed.
  • Leave step 1 before profile GET finishes, return, save — should succeed after load.
  • New webinar: save still blocked until each catalogue field is explicitly chosen (product or No).
  • Edit event with real PPN values from API: save without re-opening pickers — should succeed.

Made with Cursor

Merge publishing-profile metadata with catalogue acknowledgments in one
update when possible, backfill when the catalogue loads later, reset
profile state on eventId change, and merge profileId from ESP association
when the nested profile omits it (follow-up to explicit PPN selection).

Co-authored-by: Cursor <cursoragent@cursor.com>
@qiyundai qiyundai merged commit 84f21c9 into dev May 20, 2026
3 checks passed
@qiyundai qiyundai mentioned this pull request May 29, 2026
5 tasks
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.

2 participants