Skip to content

fix(editorApi): fix callbacks on editor events#8517

Merged
mejo- merged 1 commit intomainfrom
fix/editor_api_emit_events
Apr 27, 2026
Merged

fix(editorApi): fix callbacks on editor events#8517
mejo- merged 1 commit intomainfrom
fix/editor_api_emit_events

Conversation

@mejo-
Copy link
Copy Markdown
Member

@mejo- mejo- commented Apr 27, 2026

The new Editor.js wrapper component that got introduced in the indexedb PR broke event propagation from Editor.vue component to editorApi consumers.

Fixes loading editor and updating read-only content in Collectives.

The resolved emit was a leftover (dead code) so I removed it.

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits

@mejo- mejo- self-assigned this Apr 27, 2026
@mejo- mejo- added bug Something isn't working 3. to review regression labels Apr 27, 2026
@codecov
Copy link
Copy Markdown

codecov Bot commented Apr 27, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.

📢 Thoughts on this report? Let us know!

@mejo- mejo- force-pushed the fix/editor_api_emit_events branch from 7a5e8b6 to ac8240f Compare April 27, 2026 10:57
Comment thread src/components/Editor.js Outdated
The new Editor.js wrapper component that got introduced in the indexedb
PR broke event propagation from Editor.vue component to editorApi
consumers.

Fixes loading editor and updating read-only content in Collectives.

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the fix/editor_api_emit_events branch from ac8240f to 62ce44d Compare April 27, 2026 12:22
@mejo- mejo- requested a review from silverkszlo April 27, 2026 12:22
@mejo- mejo- merged commit 038120d into main Apr 27, 2026
63 of 67 checks passed
@mejo- mejo- deleted the fix/editor_api_emit_events branch April 27, 2026 12:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

3. to review bug Something isn't working regression

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants