-
Notifications
You must be signed in to change notification settings - Fork 645
Redirect /install to doc.crates.io #1035
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
We're planning on deprecating doc.crates.io in the near-ish future; I think it's probably a better idea to just move the content over here, and redirect the other way. |
@steveklabnik I was thinking of updating the redirect later to the "Installation" chapter of the book, whenever the URL is stable. AFAIU, we want to keep that chapter in the Cargo Manual mdbook anyway, and it's easier to keep them in sync if both are in the What do you think? |
Uh, and the reason it's redirecting to the homepage of |
Ugh you know what, I also got confused with the policies vs install bit here, my bad. Sigh.
Yeah this seems good. Sorry for the noise! |
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.
Just a few questions/suggestions!
app/templates/install.hbs
Outdated
Rust comes with Cargo by default, so the easiest way to get Cargo is to | ||
<a href='http://www.rust-lang.org/install.html'>install Rust</a> via | ||
the official binary installers. | ||
Redirecting you to <code>https://crates.io/</code>… |
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.
Shouldn't this say "Redirecting you to http://doc.crates/io
"? Since that's where the redirect is actually going...?
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.
Right. Updated.
|
||
export default Route.extend({ | ||
redirect() { | ||
window.location = 'http://doc.crates.io/'; |
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.
Should this be https
? Or protocol-relative, so just //doc.crates.io
?
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.
a1e664e
to
d0084a1
Compare
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.
Thanks! Addressed the comment.
|
||
export default Route.extend({ | ||
redirect() { | ||
window.location = 'http://doc.crates.io/'; |
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.
app/templates/install.hbs
Outdated
Rust comes with Cargo by default, so the easiest way to get Cargo is to | ||
<a href='http://www.rust-lang.org/install.html'>install Rust</a> via | ||
the official binary installers. | ||
Redirecting you to <code>https://crates.io/</code>… |
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.
Right. Updated.
The content of the `install` page here is duplicate of `cargo`'s *Installation* page/section. Since it's not *registry*-specific feature, we better just redirect to the original source. The content of the page is already merged into <http://doc.crates.io/> and is live. Closes <rust-lang#1029>
Much better, thanks!! bors: r+ |
1035: Redirect /install to doc.crates.io r=carols10cents The content of the `install` page here is duplicate of `cargo`'s *Installation* page/section. Since it's not *registry*-specific feature, we better just redirect to the original source. The content of the page is already merged into <http://doc.crates.io/> and is live. See <#1029>
Build succeeded |
The content of the
install
page here is duplicate ofcargo
'sInstallation page/section. Since it's not registry-specific feature,
we better just redirect to the original source.
The content of the page is already merged into http://doc.crates.io/
and is live.
See #1029