Skip to content

fix: persist block media selections#645

Open
bobbwal wants to merge 2 commits intoSonicJs-Org:mainfrom
bobbwal:fix/block-media-persist
Open

fix: persist block media selections#645
bobbwal wants to merge 2 commits intoSonicJs-Org:mainfrom
bobbwal:fix/block-media-persist

Conversation

@bobbwal
Copy link
Contributor

@bobbwal bobbwal commented Feb 6, 2026

Description

Ensure media selections inside block/object fields persist by dispatching input/change events when media values update, and add E2E coverage.

Fixes #644

Changes

  • Dispatch input/change events for media field updates (select/remove/clear) to keep structured fields in sync.
  • Add E2E coverage for media persistence in content blocks.

Testing

  • npx playwright test tests/e2e/53-block-media-persist.spec.ts --config tests/playwright.config.ts

Unit Tests

  • Added/updated unit tests
  • All unit tests passing

E2E Tests

  • Added/updated E2E tests
  • All E2E tests passing

Checklist

  • Code follows project conventions
  • Tests added/updated and passing
  • Type checking passes
  • No console errors or warnings
  • Documentation updated (if needed)

Generated with Claude Code in Conductor

@bobbwal bobbwal marked this pull request as draft February 9, 2026 17:37
@bobbwal bobbwal marked this pull request as ready for review February 10, 2026 15:49
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.

Bug: Media selection in content blocks doesn’t persist after save

1 participant