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

fix Netlify redirects #5177

Merged
merged 5 commits into from Jan 21, 2023
Merged

fix Netlify redirects #5177

merged 5 commits into from Jan 21, 2023

Conversation

tjenkinson
Copy link
Member

@tjenkinson tjenkinson commented Jan 21, 2023

This PR will...

Fix Netlify root redirect. The problem was the second line to setup the api-docs redirect was replacing the file contents instead of appending (>>).

Second reason it wasn't working is because netlify won't redirect by default if a file already exists, so now we delete the index file.

Why is this Pull Request needed?

New commit deployments and https://hls-js-dev.netlify.com/ go to a 404 at the root and api docs redirect doesn’t work

@tjenkinson tjenkinson marked this pull request as ready for review January 21, 2023 11:42
@tjenkinson tjenkinson added the CI label Jan 21, 2023
@tjenkinson tjenkinson merged commit 6a7524f into master Jan 21, 2023
@tjenkinson tjenkinson deleted the fix-netlify-redirects branch January 21, 2023 16:17
robwalch pushed a commit that referenced this pull request Jan 26, 2023
* fix Netlify redirects

* remove api docs redirect

* Revert "remove api docs redirect"

This reverts commit e86af6f.

* remove the index file from api docs

* also redirect index.html
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.

None yet

2 participants