Skip to content
This repository has been archived by the owner on Oct 14, 2023. It is now read-only.

binder link: embedded hyperlinks broken #781

Closed
mathematicalmichael opened this issue Oct 10, 2019 · 4 comments 路 Fixed by #793
Closed

binder link: embedded hyperlinks broken #781

mathematicalmichael opened this issue Oct 10, 2019 · 4 comments 路 Fixed by #793

Comments

@mathematicalmichael
Copy link
Contributor

馃悶 Problem

Clicking on the binder link in the README leads to a page with broken hyperlinks. Cannot navigate to examples.

The cached version that shows up during loading does work, though.

@astrojuanlu
Copy link
Member

Hi @mathematicalmichael, thanks for your interest in poliastro! You are right, now the links are not being rendered:

Screenshot_2019-10-14 index

I am sure this used to work at some point, we will have to consider some alternatives.

@shreyasbapat
Copy link
Member

I faced this similar problem! For me, replacing all the spaces in the name with an '_' (Underscore) worked!

@mathematicalmichael
Copy link
Contributor Author

mathematicalmichael commented Oct 29, 2019

I faced this similar problem! For me, replacing all the spaces in the name with an '_' (Underscore) worked!

oh yeah, that's definitely it.
not sure if markdown takes escape \ into account, but this is exactly why I err on the side of caution and never use spaces in filenames.

trying %20 instead of _ just to see if that'll do the trick

mathematicalmichael added a commit to mathematicalmichael/poliastro that referenced this issue Oct 29, 2019
addresses poliastro#781 (hopefully... did this manually in the GitHub editor)
@shreyasbapat
Copy link
Member

Minor change required to finally close this!

shreyasbapat added a commit to shreyasbapat/poliastro that referenced this issue Nov 2, 2019
astrojuanlu added a commit that referenced this issue Nov 3, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants