-
Notifications
You must be signed in to change notification settings - Fork 230
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 redirects #649
Fix redirects #649
Conversation
Since we're now serving the website fully statically, we need to use meta redirects rather than relying on our server configuration. Closes #650
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.
LGTM just one potentially minor comment/question
|
||
Dir['source/documentation/breaking-changes/**'].each do |file| | ||
basename = File.basename(file).gsub(/\..*/, '') | ||
redirect "d/#{basename}.html", to: "/documentation/breaking-changes/#{basename}" |
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.
Only nitpick is that before some redirects were any /d/whatever to /doc.../break.../whatever, regardless of whether the url was known or not, and now it redirects on known URLs only
(Same on others)
I assume that's intended because it would throw 404 regardless?
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.
These redirects are actually compiled down to HTML pages with <meta http-equiv="refresh">
tags, so we unfortunately can't do wildcard redirects anymore. Otherwise I'd keep them if I could.
Since we're now serving the website fully statically, we need to use meta redirects rather than relying on our server configuration. Closes sass#650
Since we're now serving the website fully statically, we need to use meta redirects rather than relying on our server configuration. Closes sass#650
Since we're now serving the website fully statically, we need to use
meta redirects rather than relying on our server configuration.