-
Notifications
You must be signed in to change notification settings - Fork 21
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
Redirect static base URLs to a helpful landing page #77
Conversation
Seems to work, try e.g. https://deploy-preview-77--optimade-providers.netlify.app/index-metadbs/tcod/ |
11817ff
to
19ba0a0
Compare
19ba0a0
to
7f30e5f
Compare
Think this is ready @CasperWA! |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Great! Thanks @ml-evs !
I have only one suggested addition.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Huzzah 🚀
This PR is a potential solution for #76, by redirecting the static base URLs served here to a semi-informative landing page, rather than 404'ing.
Unfortunately netlify can't do redirects based on
Accept
headers, so serving a JSON response is not possible (unless we get rid of this HTML response). The HTML response mimics the way most providers are handling their base URLs, so I don't think there is an issue here.