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

Fixed links having http:// added to the beginning in certain cases. #5323

Merged
merged 3 commits into from
Apr 13, 2024

Conversation

Mm2PL
Copy link
Collaborator

@Mm2PL Mm2PL commented Apr 13, 2024

Display the original link text (not the one with the protocol)

Fixes #5322. @pajlada please hold release 2.5.0 until the issue is fixed. This is a really simple fix. I don't think I've made mistakes in this?

(not the one with the protocol)
@Mm2PL Mm2PL requested a review from pajlada April 13, 2024 00:54
@Mm2PL Mm2PL changed the title Display the original link text Fixed links having http:// added to the beginning in certain cases. Apr 13, 2024
@pajlada pajlada added this to the Investigate before 2.5.0 milestone Apr 13, 2024
@pajlada
Copy link
Member

pajlada commented Apr 13, 2024

Not sure how to reproduce, could you post some message that is broken?

@pajlada
Copy link
Member

pajlada commented Apr 13, 2024

felanbird helped me disable the "lowercase links" option and now I can see it

@pajlada pajlada enabled auto-merge (squash) April 13, 2024 07:56
@pajlada pajlada merged commit 7285f08 into master Apr 13, 2024
17 checks passed
@pajlada pajlada deleted the fix/fix-protocol-shit branch April 13, 2024 08:18
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.

Non-lowercased links always add a protocol
2 participants