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: message-parser links #28735

Merged
merged 2 commits into from Apr 1, 2023
Merged

fix: message-parser links #28735

merged 2 commits into from Apr 1, 2023

Conversation

hugocostadev
Copy link
Contributor

Proposed changes (including videos or screenshots)

When sending URLs with certain characters, the hyperlink gets cut off.

For example:

https://www.nimblepcm.com/(W(601))/Main?ScreenId=GI000027

URL gets cut off after "(601"

That doesn't happen with all URLs.

Issue(s)

Steps to test or reproduce

Further comments

TC-538

@codecov
Copy link

codecov bot commented Mar 31, 2023

Codecov Report

Merging #28735 (1c965ef) into develop (1aad44b) will decrease coverage by 0.04%.
The diff coverage is n/a.

Impacted file tree graph

@@             Coverage Diff             @@
##           develop   #28735      +/-   ##
===========================================
- Coverage    45.25%   45.22%   -0.04%     
===========================================
  Files          738      738              
  Lines        14461    14461              
  Branches      2236     2236              
===========================================
- Hits          6545     6540       -5     
- Misses        7611     7616       +5     
  Partials       305      305              
Flag Coverage Δ
e2e 45.18% <ø> (-0.04%) ⬇️

Flags with carried forward coverage won't be shown. Click here to find out more.

@ggazzo ggazzo merged commit 97c5598 into develop Apr 1, 2023
34 checks passed
@ggazzo ggazzo deleted the fix/message-parser-links branch April 1, 2023 14:15
gabriellsh added a commit that referenced this pull request Apr 4, 2023
…avatarEmoji

* 'develop' of github.com:RocketChat/Rocket.Chat: (63 commits)
  fix: Messages jumping after reaction (#28770)
  fix: Quotes chain off by one error in quote chain limit settings (#28281)
  docs: update documentation (#28719)
  chore: /help notifying the shortcuts in multiple messages (#28765)
  refactor: remove Users from fibers 14 (#28780)
  regression(file-upload:s3): Fix missing store import (#28781)
  refactor: remove Users from fibers 13 (#28772)
  refactor: remove Users from fibers 12 (#28767)
  ci: revert Variable Setup changes (#28754)
  feat: auto link custom domain (#28501)
  refactor: `Rooms` model 4/4 (#28742)
  regression: invalid callback callAsync (#28766)
  refactor: remove Users from fibers 11 (#28740)
  refactor: Convert our Upload Code to TS (#28711)
  refactor: Move callbacks to ts 1X (#28710)
  refactor: Remove Users from fibers 10 (#28737)
  fix: message-parser links (#28735)
  refactor(client): Miscelaneous (#28732)
  refactor: `Rooms` model 3/4 (#28736)
  refactor: `Rooms` model 2X (#28715)
  ...
@sampaiodiego sampaiodiego mentioned this pull request May 16, 2023
Kishn0109 pushed a commit to Kishn0109/Rocket.Chat that referenced this pull request May 26, 2023
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.

None yet

3 participants