Skip to content

Fix generation of .livemd paths to handle omitted '.html' #1741

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

Merged
merged 1 commit into from
Jul 18, 2023
Merged

Fix generation of .livemd paths to handle omitted '.html' #1741

merged 1 commit into from
Jul 18, 2023

Conversation

josecfreittas
Copy link
Contributor

Some HTTP servers redirect and omit the .html extension when you go to an html page, hosting a generated doc on one of these makes the “Run in Livebook” button break because it doesn’t add the .livemd extension to the link. This PR fixes it making the .html optional in the replacement.

@josecfreittas josecfreittas changed the title Fix livebook paths to handle omitted '.html' Fix generation of .livemd paths to handle omitted '.html' Jul 18, 2023
@josevalim josevalim merged commit be43c5a into elixir-lang:main Jul 18, 2023
@josevalim
Copy link
Member

💚 💙 💜 💛 ❤️

@josecfreittas josecfreittas deleted the josecfreittas-patch-1 branch July 18, 2023 22:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants