Skip to content

MWPW-203849: POC: Add concurrent editing presence banner to Studio fragment editors - #1178

Draft
seanchoi0 wants to merge 1 commit into
mainfrom
MWPW-203849
Draft

MWPW-203849: POC: Add concurrent editing presence banner to Studio fragment editors#1178
seanchoi0 wants to merge 1 commit into
mainfrom
MWPW-203849

Conversation

@seanchoi0

Copy link
Copy Markdown
Contributor

Shows a yellow warning banner when multiple authors are editing the same fragment at the same time. The banner disappears automatically when the other editor closes the fragment.

Resolves https://jira.corp.adobe.com/browse/MWPW-203849
QA Checklist: https://wiki.corp.adobe.com/display/adobedotcom/M@S+Engineering+QA+Use+Cases

Implementation

  • New ConcurrentEditController (Lit ReactiveController) in studio/src/reactivity/concurrent-edit-controller.js manages WebSocket lifecycle to a Cloudflare Workers Durable Object (wss://mas-presence.mas-presence.workers.dev)

  • Added to both editor-panel.js (Collections side panel) and mas-fragment-editor.js (Card full-page editor)

  • WebSocket connects per fragment ID on edit open, disconnects on close

  • Banner shows when 2+ connections are active on the same fragment; hides when count drops to 1

  • Deduplicates by email to handle same user with multiple tabs

  • C1. Cover code with Unit Tests

  • C2. Add a Nala test (double check with #fishbags if nala test is needed)

  • C3. Verify all Checks are green (unit tests, nala tests)

  • C4. PR description contains working Test Page link where the feature can be tested

  • C5: you are ready to do a demo from Test Page in PR (bonus: write a working demo script that you'll use on Thursday, you can eventually put in your PR)

  • C.6 read your Jira one more time to validate that you've addressed all AC's and nothing is missing

Test URLs:

…t editors

Shows a warning banner when multiple authors are editing the same fragment simultaneously.
Uses a WebSocket connection to a Cloudflare Workers Durable Object that broadcasts
presence updates per fragment ID.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
@aem-code-sync

aem-code-sync Bot commented Aug 25, 2026

Copy link
Copy Markdown

Hello, I'm the AEM Code Sync Bot and I will run some actions to deploy your branch.
In case there are problems, just click the checkbox below to rerun the respective action.

  • Re-sync branch
Commits

@github-actions

Copy link
Copy Markdown

SNOW Change Request Transaction ID [STUDIO]: 0000019c1eec85a3-1f56e52a

@github-actions

Copy link
Copy Markdown

SNOW Change Request Transaction ID [IO]: 0000019c1eec85a3-1f56e594

@seanchoi0 seanchoi0 changed the title MWPW-203849: Add concurrent editing presence banner to Studio fragment editors MWPW-203849: POC: Add concurrent editing presence banner to Studio fragment editors Aug 25, 2026
@seanchoi0
seanchoi0 marked this pull request as draft August 25, 2026 14:36
@seanchoi0 seanchoi0 added do not merge PR should not be merged yet DO NOT REVIEW labels Aug 25, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

do not merge PR should not be merged yet DO NOT REVIEW

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant