Skip to content

Fix/vue3 followup - #8822

Merged
mejo- merged 3 commits into
mainfrom
fix/vue3_followup
Jul 7, 2026
Merged

mejo- merged 3 commits into
mainfrom
fix/vue3_followup

Conversation

@mejo-

@mejo- mejo- commented Jul 6, 2026

Copy link
Copy Markdown
Member

🏁 Checklist

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

🤖 AI (if applicable)

  • The content of this PR was partly generated using AI tools
  • The AI-generated content was reviewed, comprehended and tested by a human

mejo- added 2 commits July 6, 2026 18:26
This syntax seems to be supported by Vue2 but no longer by Vue3. It's
redundant anyway as the used components CollaborativeEditorApp und
MarkdownContentEditorApp already have `ref="editor-container"`.

Fixes "[Vue warn]: Missing ref owner context. ref cannot be used on
hoisted vnodes. A vnode with ref must be created inside the render
function."

Signed-off-by: Jonas <jonas@freesources.org>
…nmount

The editor might already be destroyed on unmount.

Fixes "[tiptap error]: The editor view is not available. Cannot access
view['dom']. The editor may not be mounted yet."

Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- self-assigned this Jul 6, 2026
@mejo- mejo- added the bug Something isn't working label Jul 6, 2026
@github-project-automation github-project-automation Bot moved this to 🧭 Planning evaluation (don't pick) in 📝 Productivity team Jul 6, 2026

@max-nextcloud max-nextcloud left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!

Signed-off-by: Jonas <jonas@freesources.org>
@mejo-
mejo- force-pushed the fix/vue3_followup branch from 3433c72 to e301ccf Compare July 7, 2026 07:50
@mejo-
mejo- merged commit a89e2c3 into main Jul 7, 2026
61 checks passed
@mejo-
mejo- deleted the fix/vue3_followup branch July 7, 2026 08:28
@github-project-automation github-project-automation Bot moved this from 🧭 Planning evaluation (don't pick) to ☑️ Done in 📝 Productivity team Jul 7, 2026
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

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants