-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Also check latest release for Custom 404 page #8089
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
Comments
This makes sense to me. We would need to think a little more about this since our 404 checks are already super slow (we hit storage backend multiple times) and this sometimes causes us some problems. Adding an extra check here will definitely increase the times we will hit it. I already attempted to fix this problem in #6797 but we finally took another direction. The issue is not fixed and that PR would definitely help, but it also increases complexity 🤷
This is something we definitely should do in https://docs.readthedocs.io/en/stable/hosting.html#custom-not-found-404-pages -- cc @astrojuanlu |
Thanks and understandable. Another additional (or separate) improvement for consideration is to allow admins to choose which fallback should take priority between stable and latest. Since the fallback is global, that choice could be useful |
We clarified the documentation about the current strategy. I propose to leave the issue open to discuss any changes. |
The current 404 resolution strategy is:
Would it be possible to fallback to the latest version if the default version doesn't have a 404 page? Our default version is stable, but the 404 is on latest which made me wonder why it wasn't working in all cases.
It would also help if the resolution strategy was documented.
The text was updated successfully, but these errors were encountered: