search

kquick / Thespian

Python Actor concurrency library
MIT License
189 stars 24 forks source link

Document links still broken #4

Closed kquick closed 7 years ago

kquick commented 7 years ago

The initial problem with the document links described in https://github.com/godaddy/Thespian/issues/41 seems to be resolved, but internal links are still not working as targets: copying a link and trying to open it in a new window always just opens the last section of the document ("Footnotes" for the "Using Thespian" document).

tjkluska commented 7 years ago

Should be http://thespianpy.com/doc/using

kquick commented 7 years ago

The documentation links are now fixed, but I did need to adjust the static references: any URL reference to a specific documentation location should change "#h:..." to "#hH-...". The problem turns out to be a limitation in jQuery (http://api.jquery.com/category/selectors/) which required escaping of the : character in the references.

Fixed as of commit ed10ecf619200a2debba865ff33a2f50a50c327e.

fasiha commented 6 years ago

The link to the “Using Thespian” doc at the bottom of Quick Start still points to http://thespian.py/doc/using instead of http://thespianpy.com/doc/using ?

kquick commented 6 years ago

Fixed. Thank you for the report, @fasiha !