Skip to content

fix the base href in 404.html #746

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

Closed
wants to merge 1 commit into from
Closed

fix the base href in 404.html #746

wants to merge 1 commit into from

Conversation

Fil
Copy link
Contributor

@Fil Fil commented Feb 10, 2024

https://observablehq.com/framework/404
https://observablehq.com/framework/examples/chess/404 (etc.)

closes #721
alternative to #194

Doing this in CD means we're not tying our hands with #194 ; if in the future we do #194 we'd replace this with the relevant build option.

(Tested locally, but it might break in a github action.)

(sed option -i. means in-place replacement with no backup file)

@Fil Fil requested review from mbostock and mythmon February 10, 2024 17:12
@mbostock mbostock mentioned this pull request Feb 10, 2024
@mbostock mbostock closed this Feb 10, 2024
@Fil Fil deleted the fil/sed-404 branch February 10, 2024 19:48
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.

404 page doesn’t render correctly on a subpath
2 participants