-
-
Notifications
You must be signed in to change notification settings - Fork 13
fix: redirect latest version to stable #774
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
Conversation
✅ Deploy Preview for stackable-docs ready!
To edit notification comments on pull requests, go to your Netlify project configuration. |
Redirects working in the preview |
This was actually removed previously, because it caused other issues. See #678. |
Would it make sense to look into the other issues one more time then (sorry don't know about the linked issue yet)? Because as far as I understood this is the solution to having the redirects and otherwise the strategy defaults to Edit: We decided to take another look at the previous problems |
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.
Approved, under the condition that this will need to be reverted/changed to make #678 work as well.
The linked issue doesn't seem to be a problem anymore. The redirect works as expected. |
Research Results
Netlify was only processing one redirect rule. The one processed here is found in
netlify.toml
:There was no
_redirects
file as part of the prod deployment in Netlify, which could contain additional redirects.After adding the change in the PR, locally, a
_redirects
file was generated, which contains the following redirects listed below. The relevant redirect for this fix is the top one, but deploying this might also add the other redirects which might have been missing before also.