Skip to content

[stable33] fix(links): allow to pass custom link handler into editor and use it#8581

Merged
mejo- merged 4 commits into
stable33from
backport/8571/stable33
May 13, 2026
Merged

[stable33] fix(links): allow to pass custom link handler into editor and use it#8581
mejo- merged 4 commits into
stable33from
backport/8571/stable33

Conversation

@backportbot
Copy link
Copy Markdown

@backportbot backportbot Bot commented May 13, 2026

Backport of #8571

Todo

  • Review and resolve any conflicts
  • Review and verify the backported changes
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Signed-off-by: Jonas <jonas@freesources.org>
mejo- added 3 commits May 13, 2026 15:31
The return of the link handler 👻

Required as we want to pass a link handler from Collectives to be used
for the "Open link" button and when opening a link via Ctrl-click.

Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
Signed-off-by: Jonas <jonas@freesources.org>
@mejo- mejo- force-pushed the backport/8571/stable33 branch from 823c030 to d864eb6 Compare May 13, 2026 13:36
@mejo- mejo- marked this pull request as ready for review May 13, 2026 13:37
@mejo- mejo- requested a review from silverkszlo as a code owner May 13, 2026 13:37
@codecov
Copy link
Copy Markdown

codecov Bot commented May 13, 2026

Codecov Report

❌ Patch coverage is 60.00000% with 16 lines in your changes missing coverage. Please review.

Files with missing lines Patch % Lines
src/plugins/links.ts 60.00% 8 Missing and 2 partials ⚠️
src/components/Editor/PreviewOptions.vue 0.00% 4 Missing ⚠️
src/components/Link/LinkBubbleView.vue 75.00% 1 Missing ⚠️
src/helpers/links.js 0.00% 1 Missing ⚠️

📢 Thoughts on this report? Let us know!

@mejo- mejo- merged commit 936e1af into stable33 May 13, 2026
64 checks passed
@mejo- mejo- deleted the backport/8571/stable33 branch May 13, 2026 14:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant