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

Flyout links incorrect on Mkdocs projects #1917

Closed
dnrce opened this issue Jan 12, 2016 · 21 comments
Closed

Flyout links incorrect on Mkdocs projects #1917

dnrce opened this issue Jan 12, 2016 · 21 comments
Labels
Bug A bug
Milestone

Comments

@dnrce
Copy link

dnrce commented Jan 12, 2016

GitHub links in the sidebar footer contain an extraneous path. For example, on http://tracks.readthedocs.org/en/latest/upgrading/ the "View" link points to https://github.com/TracksApp/tracks/blob/master/home/docs/checkouts/readthedocs.org/user_builds/tracks/checkouts/latest/doc/Upgrading.md.

@agjohnson agjohnson changed the title GitHub links include "checkouts" path [MkDocs] Flyout links incorrect on Mkdocs projects Jan 12, 2016
@marcelstoer
Copy link
Contributor

#1480 was closed even though the root cause seems to be the same. Yet, the flyout links are still incorrect.

@sahilTakiar
Copy link

I was trying to fix this and believe I found the cause, I was able to get a working version for a MkDocs based project.

More info here: 98b5d34

@sahilTakiar
Copy link

@agjohnson any thoughts on my comment in 98b5d34

I have tested the change locally using the directions here: https://docs.readthedocs.org/en/latest/install.html#installing-read-the-docs and it works for a MkDocs and Sphinx based projects.

@sahilTakiar
Copy link

@ericholscher do you have any thoughts on this? If I have a fix that is tested, so I can create the PR. Want to get this fixed as we just went live with our ReadTheDocs integration.

@ericholscher
Copy link
Member

@sahilTakiar Definitely send the PR over!

@kenorb
Copy link

kenorb commented Apr 19, 2016

The same here.

Steps to reproduce:

  1. Go to this page.
  2. Click Read the Docs in the bottom, then View or Edit.

The links are like:

https://github.com/geerlingguy/drupal-vm/blob/master/home/docs/checkouts/readthedocs.org/user_builds/drupal-vm/checkouts/latest/docs/index.md
https://github.com/geerlingguy/drupal-vm/edit/master/home/docs/checkouts/readthedocs.org/user_builds/drupal-vm/checkouts/latest/docs/index.md

which aren't valid.

@laughinghan
Copy link

ericholscher added a commit that referenced this issue Mar 5, 2017
(#1917) Fixing Flyout Links for MkDocs Projects
@marcelstoer
Copy link
Contributor

#2066 was merged hours ago and thus this should now be closed I guess.

@dnrce
Copy link
Author

dnrce commented Mar 5, 2017

Thanks @marcelstoer. I just rebuilt and the issue persists so I don't believe this should be closed until the change is deployed.

@ericholscher
Copy link
Member

Just deployed this, so it should hopefully be fixed.

@dnrce
Copy link
Author

dnrce commented Mar 6, 2017

It appears to be. Thanks @sahilTakiar and @ericholscher!

@dnrce dnrce closed this as completed Mar 6, 2017
@marcelstoer
Copy link
Contributor

marcelstoer commented Mar 6, 2017

It works for the OP's project at http://tracks.readthedocs.org but it doesn't work for any other project listed here in the comments: http://docs.drupalvm.com, http://mathquill.readthedocs.io and http://nodemcu.readthedocs.io (mine) still have this odd ".../docs/checkouts/readthedocs.org/..." URLs. What's the difference between them? I just built mine again to be sure.

@dnrce dnrce reopened this Mar 6, 2017
@markekraus
Copy link

same for me. The view and edit URLs have the same '.../docs/checkouts/readthedocs.org/...' in them

I thought maybe it was related to having edit_uri set in mckdocs.yml, but removing it and rebuilding seems to make no difference. @marcelstoer doesn't even have the set in their mkdcos.yml.

@PowerKiKi
Copy link
Contributor

edit_uri is a feature that appeared in mkdocs 0.16 and that would help in solving that I think. But it seems RTFD is still using mkdocs 0.15 according to my build logs.

Is an upgrade to mkdocs 0.16 planed ? Or is it something that could be contributed by someone relatively easily ?

fv3rdugo added a commit to UPC/ravada that referenced this issue May 24, 2017
I see this: readthedocs/readthedocs.org#1917
The problem is the links to github.
@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 Jun 6, 2018

I didn't know that this issue was already reported, I'm fixing this in #3525

@ryanpitts
Copy link

Sorry to ping in a closed issue, but I'm seeing the same broken "View/Edit on GitHub" links in this project, which is using Mkdocs for the build: http://fieldguide.opennews.org/en/latest/. Both URLs have the ".../docs/checkouts/readthedocs.org/..." path stuff appended.

I just triggered a fresh build to see if that would clear things up (it had been a while), but no luck. Is there a setting I need to update?

@stsewd
Copy link
Member

stsewd commented Jan 5, 2019

Links are correct for me

@ryanpitts
Copy link

Hmmm. I hoped maybe it was just a caching thing so I cleared that out, but the links are still broken for me. (See URL at bottom of screenshot)

screen shot 2019-01-04 at 5 34 06 pm

@PowerKiKi
Copy link
Contributor

Links are correct for me too. Might be CDN related cache?

@stsewd
Copy link
Member

stsewd commented Jan 5, 2019

Yeah, I think so, CDN or ISP caching

@ryanpitts
Copy link

Hey! Sure enough, those links are correct for me now too. (I promise I tried clearing caches, multiple browsers, etc etc.) Thanks for the response!

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

No branches or pull requests