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

Corrected permalink #5949

Merged
merged 2 commits into from Mar 21, 2017

Conversation

Projects
None yet
3 participants
@lukebro
Contributor

lukebro commented Mar 12, 2017

Removed the paragraph telling a user to visit the navigations page to learn how to build more robust navigation.

The permalink was broken since Navigation no longer exists and no other suitable substitute (closest being ./permalinks) fits the description.

Removed navigation paragraph
Removed the paragraph telling a user to visit the navigations page to learn how to build more robust navigation.

The permalink was broken since Navigation no longer exists and no other suitable substitute (closest being ./permalinks) fits the description.
@DirtyF

You're right we should have updated this link when navigation was moved to /tutorials/navigation/.
Feel free to fix it.

Corrected permalink
Fixed the permalink to navigation page since it was moved to under tutorials.
@lukebro

This comment has been minimized.

Show comment
Hide comment
@lukebro

lukebro Mar 12, 2017

Contributor

@DirtyF 👍 all set

Contributor

lukebro commented Mar 12, 2017

@DirtyF 👍 all set

@lukebro lukebro changed the title from Removed navigation paragraph to Corrected permalink Mar 12, 2017

@DirtyF

DirtyF approved these changes Mar 13, 2017

@DirtyF

This comment has been minimized.

Show comment
Hide comment
@DirtyF

DirtyF Mar 21, 2017

Member

@jekyllbot: merge +docs

Member

DirtyF commented Mar 21, 2017

@jekyllbot: merge +docs

@jekyllbot jekyllbot merged commit 061aa40 into jekyll:master Mar 21, 2017

1 of 2 checks passed

continuous-integration/appveyor/pr AppVeyor build failed
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details

jekyllbot added a commit that referenced this pull request Mar 21, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment