Skip to content

View / Edit doesn't work #3515

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

Closed
sergei-bondarenko opened this issue Jan 15, 2018 · 5 comments
Closed

View / Edit doesn't work #3515

sergei-bondarenko opened this issue Jan 15, 2018 · 5 comments
Labels
Support Support question
Milestone

Comments

@sergei-bondarenko
Copy link

sergei-bondarenko commented Jan 15, 2018

Details

Expected Result

Click on v:latest in the right bottom corner -> On GitHub -> View / Edit.
It should open https://github.com/HelloZeroNet/Documentation/blob/master/docs/faq.md

Actual Result

404 Error.

P.S. Please don't close as a duplicate of #1422 or #1483, because it's not fixed.

@stsewd
Copy link
Member

stsewd commented Jan 16, 2018

I think this issue is related to Mkdocs only, with sphinx works fine.

I was able to replicate this on my local instance, looks like the "Edit" url is taken from the mkdocs.yml file.

And also I noted that the footer url (from the floating v) is incorrect https://github.com/HelloZeroNet/Documentation/blob/master/home/docs/checkouts/readthedocs.org/user_builds/zeronet/checkouts/latest/docs/.md

@stsewd
Copy link
Member

stsewd commented Jan 16, 2018

I think the top edit button is related to the theme (not sure, I was unable to find that code), the link on the footer is definitely related to the code base.

@stsewd
Copy link
Member

stsewd commented Jan 17, 2018

After investigating more on this issue, I found you are using a custom theme (based on rtd). So, the real problem is on the templates, isn't a RTD bug.

For the "edit" link you need to update this file https://github.com/HelloZeroNet/Documentation/blob/master/readthedocs/breadcrumbs.html and inject the appropriates values (see http://www.mkdocs.org/about/release-notes/#context-variables).

About the second problem, when #3525 is merged you need to define the global variable READTHEDOCS_DATA['page'] (something like this https://github.com/HelloZeroNet/Documentation/blob/master/readthedocs/base.html#L25-L29, but must be after this lines https://github.com/HelloZeroNet/Documentation/blob/master/readthedocs/base.html#L38-L40). And everything would work properly.

@RichardLitt RichardLitt added Support Support question and removed Bug A bug labels Jan 17, 2018
@agjohnson agjohnson added this to the Mkdocs milestone Apr 10, 2018
@agjohnson agjohnson removed the Mkdocs label Apr 10, 2018
@stsewd
Copy link
Member

stsewd commented Apr 21, 2018

@grez911 I just see you deleted the whole custom theme, about the edit link please see #3991 (comment)

@stsewd
Copy link
Member

stsewd commented May 10, 2018

Closing in favor of #3991

@stsewd stsewd closed this as completed May 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Support Support question
Projects
None yet
Development

No branches or pull requests

5 participants