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(mentions): encode user id in mentions URLs #4588

Merged
merged 2 commits into from
Jul 27, 2023
Merged

Conversation

mejo-
Copy link
Member

@mejo- mejo- commented Jul 27, 2023

Whitespaces in the URL of mentions break markdown parsing, which leads to unrendered mentions for users with a whitespace in their userId.

Fixes: #4157

Todo

  • Publish new NPM package for Collectives after PR got merged

🏁 Checklist

  • Code is properly formatted (npm run lint / npm run stylelint / composer run cs:check)
  • Sign-off message is added to all commits
  • Tests (unit, integration and/or end-to-end) passing and the changes are covered with tests
  • Documentation (README or documentation) has been updated or is not required

Whitespaces in the URL of mentions break markdown parsing, which leads
to unrendered mentions for users with a whitespace in their userId.

Fixes: #4157

Signed-off-by: Jonas <[email protected]>
@mejo- mejo- force-pushed the fix/mentions_encode_userid branch from 814ed3e to 63b05d2 Compare July 27, 2023 09:09
@cypress
Copy link

cypress bot commented Jul 27, 2023

1 flaky tests on run #11326 ↗︎

0 149 2 0 Flakiness 1

Details:

fix(mentions): encode user id in mentions URLs
Project: Text Commit: dcd9bd673a
Status: Passed Duration: 03:33 💡
Started: Jul 27, 2023 9:21 AM Ended: Jul 27, 2023 9:25 AM
Flakiness  cypress/e2e/files.spec.js • 1 flaky test

View Output Video

Test Artifacts
Text and server mimetypes > handle plaintext in a pre tag Output Screenshots

This comment has been generated by cypress-bot as a result of this project's GitHub integration settings.

@max-nextcloud
Copy link
Collaborator

/compile

@max-nextcloud
Copy link
Collaborator

/backport 63b05d2 to stable27

@max-nextcloud
Copy link
Collaborator

/backport 63b05d2 to stable26

Signed-off-by: nextcloud-command <[email protected]>
@mejo- mejo- merged commit e96d170 into main Jul 27, 2023
30 checks passed
@delete-merged-branch delete-merged-branch bot deleted the fix/mentions_encode_userid branch July 27, 2023 09:33
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.

User mention bubble broken in RichTextReader for users with space in username
3 participants