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

Support relative uri reference in broadcast markdown description #1464

Merged
merged 1 commit into from
Feb 19, 2025

Conversation

julien4215
Copy link
Contributor

Opening a url might fail in the description of a broadcast overview if it is a relative url (e.g. /fide/1503014/-). This PR fixes it by using Uri.resolve

@veloce
Copy link
Contributor

veloce commented Feb 19, 2025

So we are sure all the broadcast descriptions link to lichess.org?

@julien4215
Copy link
Contributor Author

No but if it is relative reference we want to use lichess.org as the base url. For example, /fide/1503014/- becomes https://lichess.org/fide/1503014/- but https://foo.org/bar stays https://foo.org/bar

@veloce veloce merged commit ddc08c2 into lichess-org:main Feb 19, 2025
1 check passed
@julien4215 julien4215 deleted the fix-broadcast-uri branch February 25, 2025 14:54
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.

2 participants