Skip to content

Conversation

jblock
Copy link
Contributor

@jblock jblock commented Jul 19, 2016

Hi! I was looking to point one of our designers to the 0.13.x docs only to find that the links were dead :( (As per #3607)

This is a small update to the README to make those links valid again. Happy to update any other legacy doc stuff as needed!

(Also, there were no code changes so I opted to not add anything to changes.md. Happy to do otherwise)

README.md Outdated
**Older Versions:**

- 0.13.x - [docs](https://github.com/reactjs/react-router/tree/0.13.x/doc) / [guides](https://github.com/reactjs/react-router/tree/0.13.x/docs/guides) / [code](https://github.com/reactjs/react-router/tree/0.13.x) / [upgrade guide](https://github.com/reactjs/react-router/blob/master/upgrade-guides/v1.0.0.md)
- 0.13.x - [docs](https://github.com/reactjs/react-router/tree/v0.13.6/doc) / [guides](https://github.com/reactjs/react-router/tree/v0.13.6/docs/guides) / [code](https://github.com/reactjs/react-router/tree/v0.13.6) / [upgrade guide](https://github.com/reactjs/react-router/blob/master/upgrade-guides/v1.0.1.md)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

v1.0.1?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That's a 404

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Fat-fingered that one. Will fix!

@taion
Copy link
Contributor

taion commented Jul 19, 2016

Thanks.

@taion
Copy link
Contributor

taion commented Jul 19, 2016

I'll merge when CI goes green.

@taion taion merged commit 7e643dc into remix-run:master Jul 19, 2016
@taion
Copy link
Contributor

taion commented Jul 19, 2016

I sorta wanna drop those links, actually.

@jblock
Copy link
Contributor Author

jblock commented Jul 19, 2016

Curious: why is that? Assuming because 0.13.x has been deprecated for ~6 months at this point :D

While they'll always be visible to those that dig through the old tags, I'd argue for having some sort of link in the Readme that points n00bs to the most recent 0.13.x tag.

@taion
Copy link
Contributor

taion commented Jul 19, 2016

It's been 8 months since 1.0.0 and 10 since 1.0.0-rc1.

@timdorr
Copy link
Member

timdorr commented Jul 19, 2016

I agree with dropping them. But let's give it until 3.0 final so folks can update their bookmarks. And we can just keep a rolling set of the 2 previous version branches.

@lock lock bot locked as resolved and limited conversation to collaborators Jan 19, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants