Skip to content

Conversation

@kevinports
Copy link
Contributor

@kevinports kevinports commented Aug 31, 2022

PR App Fix RM-2485

There's a bug with our internal link feature where if you pass a url hash of non-unicode characters as an anchor link, the href doesn't get parsed in the Anchor component.

Here's an example: https://busted-links.readme.io/docs/intercom-3

🧰 Changes

  • Ignore anchor hash when parsing internal link syntax that generates a link's path, and append the hash to the path string separately.

🧬 QA & Testing

@github-actions
Copy link

github-actions bot commented Aug 31, 2022

A review app has been launched for this PR!

🔍 Inspect the app: https://dashboard.heroku.com/apps/markdown-pr-576

🧭 Take it for a spin: https://markdown-pr-576.herokuapp.com/

  • Shut down on Wed Sep 7: Since this PR is closed, its review app has been cleaned up. 🧽

Copy link
Contributor

@kellyjosephprice kellyjosephprice left a comment

Choose a reason for hiding this comment

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

🎉

@kevinports kevinports merged commit 25c5460 into next Sep 7, 2022
@kevinports kevinports deleted the fix/special-char-in-anchor-hash branch September 7, 2022 11:04
@rafegoldberg
Copy link
Contributor

This PR was released!

🚀 Changes included in v6.52.0

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants