-
-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Flyout links incorrect on Mkdocs projects #1917
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
Comments
#1480 was closed even though the root cause seems to be the same. Yet, the flyout links are still incorrect. |
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 |
@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. |
@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. |
@sahilTakiar Definitely send the PR over! |
The same here. Steps to reproduce:
The links are like:
which aren't valid. |
Same here: http://mathquill.readthedocs.io/en/latest/ |
(#1917) Fixing Flyout Links for MkDocs Projects
#2066 was merged hours ago and thus this should now be closed I guess. |
Thanks @marcelstoer. I just rebuilt and the issue persists so I don't believe this should be closed until the change is deployed. |
Just deployed this, so it should hopefully be fixed. |
It appears to be. Thanks @sahilTakiar and @ericholscher! |
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. |
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 |
Is an upgrade to mkdocs 0.16 planed ? Or is it something that could be contributed by someone relatively easily ? |
I see this: readthedocs/readthedocs.org#1917 The problem is the links to github.
I didn't know that this issue was already reported, I'm fixing this in #3525 |
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? |
Links are correct for me |
Links are correct for me too. Might be CDN related cache? |
Yeah, I think so, CDN or ISP caching |
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! |
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.
The text was updated successfully, but these errors were encountered: