Skip to content
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

DOCS 3.0 : eliminate hard coded numbers in tutorial labels & links #4808

Merged
merged 1 commit into from Dec 21, 2015
Merged

DOCS 3.0 : eliminate hard coded numbers in tutorial labels & links #4808

merged 1 commit into from Dec 21, 2015

Conversation

SpiritLevel
Copy link
Contributor

This removes hard coded numbers as well as fixes internal links, with the aim of eliminating .htaccess rewrite rules for doc.silverstripe.org (as part of a tidy up of the rewrite rules there). Note that gurlchecker was used to identify broken internal links. Also note that multiple PRs will likely result since different doc versions have different internal link problems.

@SpiritLevel SpiritLevel changed the title DOCS 3.0 : eliminate hard coded numbers in toturial labels & links DOCS 3.0 : eliminate hard coded numbers in tutorial labels & links Dec 1, 2015
@dhensby
Copy link
Contributor

dhensby commented Dec 18, 2015

ping @camfindlay

@camfindlay
Copy link
Contributor

@dhensby this looks good to merge.

@tractorcow
Copy link
Contributor

@camfindlay the files have been renamed so they don't have the 01_, 02_, etc prefix. Are we not doing this anymore? I thought they were removed in the URLs anyway.

@camfindlay
Copy link
Contributor

@tractorcow this is an old version before we moved to that convention. The prefixes here were to pseudo do in an old version of docsviewer what the new version does now for sorting.

@tractorcow
Copy link
Contributor

Oh, I see.

tractorcow pushed a commit that referenced this pull request Dec 21, 2015
DOCS 3.0 : eliminate hard coded numbers in tutorial labels & links
@tractorcow tractorcow merged commit cbd5332 into silverstripe:3.0 Dec 21, 2015
@SpiritLevel
Copy link
Contributor Author

Thanks...@tractorcow more coming for 3.0.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

4 participants