Skip to content

fix: normalize block and section metadata serialization - #1725

Draft
hannessolo wants to merge 1 commit into
mainfrom
ewfixes
Draft

hannessolo wants to merge 1 commit into
mainfrom
ewfixes

Conversation

@hannessolo

Copy link
Copy Markdown
Contributor

Summary

  • Normalize empty block-cell paragraphs and add AEM-compatible row/cell scaffolding to header-only blocks.
  • Preserve cursor and surviving content indices.
  • Extract structured rendering-v2 section metadata without concatenating paragraphs or dropping additional links/images. Existing legacy-mode preservation stays intact.

Refs #1688, #1691, #1692, #1695. Backend initial rendering is being addressed separately in da-universal.

Validation

  • 280 targeted browser tests passed; 3 existing skips.
  • Targeted ESLint and git diff --check passed.
  • Existing dependencies reused after npm ci reported a pre-existing lockfile mismatch. Production-site behavior was not verified.

Test URLs

Address the frontend portions of #1691, #1692 and #1695 from #1688.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@aem-code-sync

aem-code-sync Bot commented Nov 12, 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

@hannessolo

hannessolo commented Nov 12, 2026 •

Copy link
Copy Markdown
Contributor Author

Scope clarification: #1704 already fixed the da-live rendering-mode gate, configuration lookup, and legacy section-metadata preservation. This draft builds on that merged work; it does not reimplement those fixes. Its remaining #1695 changes concern structured rendering-v2 values (paragraph boundaries, multiple links/images, and style classes), alongside the empty-cell/block fixes for #1691 and #1692.

Companion drafts cover paths explicitly excluded from #1704: initial server rendering in adobe/da-universal#336 and standalone Quick Edit in adobe/da-nx#992.

Backend validation: 573 tests and lint pass; local Chrome MP4 iframe playback passed through the real handlers/proxy with a mocked AEM upstream. Production-site validation remains pending.

This branch was successfully deployed

1 active deployment
ewfixes — 91610197 Deployed Nov 12, 2026 by aem-code-sync[bot]
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.

1 participant