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

Fix versioned doc handling #688

Merged
merged 3 commits into from Dec 14, 2017

Conversation

dopplershift
Copy link
Member

There were quite a few problems with relative paths that needed fixing. Closes #675.

The fixed relative path is no good. Instead, we can use the method
suggested in the templating section of the sphinx docs: the script_files
variable for the layout template. Also remove the additional loading of
jquery, since that's guaranteed to happen with our theme.
This was not properly finding either the versions.json file nor properly
linking to other versions if the user is anywhere but the main page.
@dopplershift dopplershift added Area: Docs Affects documentation Area: Infrastructure Pertains to project infrastructure (e.g. CI, linting) Type: Bug Something is not working like it should labels Dec 14, 2017
@dopplershift dopplershift added this to the 0.7 milestone Dec 14, 2017
Copy link
Contributor

@jrleeman jrleeman left a comment

Choose a reason for hiding this comment

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

LGTM

@dopplershift dopplershift merged commit 26e99ad into Unidata:master Dec 14, 2017
@dopplershift dopplershift deleted the fix-doc-versions branch December 14, 2017 20:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Area: Docs Affects documentation Area: Infrastructure Pertains to project infrastructure (e.g. CI, linting) Type: Bug Something is not working like it should
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Doc version chooser missing versions
2 participants