Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix(MarkdownContentEditor): Respect onLoaded callback passed via API #5489

Merged
merged 1 commit into from
Mar 19, 2024

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Mar 14, 2024

Emit ready once the editor is ready (Tiptap create event), which causes the onLoaded callback passed via editor API to be called.

Required by nextcloud/collectives#1150

🏁 Checklist

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

Emit `ready` once the editor is ready (Tiptap `create` event), which
causes the `onLoaded` callback passed via editor API to be called

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- added bug Something isn't working 3. to review labels Mar 14, 2024
@mejo- mejo- requested review from juliusknorr and grnd-alt March 14, 2024 17:52
@mejo-
Copy link
Member Author

mejo- commented Mar 14, 2024

/backport to stable28

@mejo-
Copy link
Member Author

mejo- commented Mar 14, 2024

/backport to stable27

@mejo- mejo- requested a review from max-nextcloud March 15, 2024 06:53
@max-nextcloud
Copy link
Collaborator

Just checked the code, not the implications.

@mejo- mejo- merged commit 1da385d into main Mar 19, 2024
59 checks passed
@mejo- mejo- deleted the fix/content_editor_onloaded branch March 19, 2024 09:59
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants