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

Play number markup in comments is not being rendered into links #898

Closed
goodger opened this issue Aug 25, 2022 · 4 comments · Fixed by #900
Closed

Play number markup in comments is not being rendered into links #898

goodger opened this issue Aug 25, 2022 · 4 comments · Fixed by #900
Assignees
Labels
Milestone

Comments

@goodger
Copy link

goodger commented Aug 25, 2022

Sabaki 0.52.1 on MacOS 12.4.

Play number markup in comments, e.g. "#55", is not being rendered into hyperlinks. This worked fine in Sabaki 0.52.0.

@apetresc apetresc added the bug label Sep 1, 2022
@apetresc apetresc self-assigned this Sep 1, 2022
@apetresc
Copy link
Member

apetresc commented Sep 4, 2022

Indeed, I can reproduce this everywhere, not just on macOS. It seems to have been broken by the react-markdown version bump, but it's very unclear to me why. Investigating now.

@apetresc
Copy link
Member

apetresc commented Sep 4, 2022

Thanks for the report! This has been fixed on the master branch - if you know how to build from master feel free to give it a spin and confirm that everything is working fine now 🙂 Otherwise it'll be released in v0.52.2 which should be ready soon.

@goodger
Copy link
Author

goodger commented Sep 5, 2022

I can confirm that play number markup is working now. Your change fixed another issue in 0.52.1 that I noticed since reporting: coordinates (like "Q3") were also not being rendered into links. Both types of links are being rendered correctly now.

I didn't actually build from master. I just "git pull"ed the changes on the master branch and used the "npm start" method to launch Sabaki.

Thanks for the fix, and thanks for Sabaki!

@apetresc
Copy link
Member

apetresc commented Sep 5, 2022

I can confirm that play number markup is working now. Your change fixed another issue in 0.52.1 that I noticed since reporting: coordinates (like "Q3") were also not being rendered into links. Both types of links are being rendered correctly now.

Awesome, glad to hear it works for you as well :) And yes, fixing this led me to a bug that was affecting all sorts of custom linking in Sabaki's Markdown.

I didn't actually build from master. I just "git pull"ed the changes on the master branch and used the "npm start" method to launch Sabaki.

That's precisely what I meant by "build from master" :) Anyway, v0.52.2 is now released, so you can just upgrade normally.

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

Successfully merging a pull request may close this issue.

2 participants