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

Update readthedocs links from .org to .io #530

Merged
merged 1 commit into from May 29, 2016

Conversation

Projects
None yet
2 participants
@bsipocz
Member

bsipocz commented May 13, 2016

This PR updates the docs links from readthedocs.org to readthedocs.io.
(a recent change, see http://blog.readthedocs.com/securing-subdomains/ )

@cdeil cdeil changed the title from Updating RTD links to Update readthedocs links from .org to .io May 13, 2016

@cdeil cdeil added this to the 0.5 milestone May 13, 2016

@cdeil cdeil self-assigned this May 13, 2016

README.rst Outdated
@@ -32,7 +32,7 @@ Status shields
:alt: Benchmarks
* .. image:: https://readthedocs.org/projects/gammapy/badge/?version=latest
:target: https://readthedocs.org/projects/gammapy/?badge=latest
:target: https://readthedocs.io/projects/gammapy/?badge=latest

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

This link is now broken?

This comment has been minimized.

@bsipocz

bsipocz May 13, 2016

Member

Oh, I didn't notice. I think it must point to the actual docs, not sure why it had the badge there. I'll amend the commit.

This comment has been minimized.

@bsipocz

bsipocz May 13, 2016

Member

also I see that you have a gammapy.org domain, so I'm changing all the gammapy.readthedocs to docs.gammapy.org.

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

also I see that you have a gammapy.org domain, so I'm changing all the gammapy.readthedocs to docs.gammapy.org.

OK, thanks!

if the docs on readthedocs show old stuff, you need to first log in `here <https://readthedocs.org/accounts/login/>`__
and then wipe it to create a fresh / clean version by hitting this URL::
http://readthedocs.org/wipe/gammapy/latest/
http://readthedocs.io/wipe/gammapy/latest/

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

Looks like this link is only available on .org, not .io?

This comment has been minimized.

@bsipocz

bsipocz May 13, 2016

Member

actually it doesn't work on the org either

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

https://readthedocs.org/wipe/gammapy/latest/ does work if you're an admin on RTD for Gammapy and logged in, so that's the correct one to put here in the docs.

This comment has been minimized.

@bsipocz

bsipocz May 13, 2016

Member

Great, it's changed back. Sorry for the noise again.

and then clicking the "wipe version" button.
You don't get a confirmation that the wipe has taken place, but you can check
`here <https://readthedocs.org/builds/gammapy/>`__ (wait a few minutes)
`here <https://readthedocs.io/builds/gammapy/>`__ (wait a few minutes)

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

Same here, this should still points to .org.

and if needed manually start a new build by going
`here <https://readthedocs.org/projects/gammapy/>`__ and clicking the "Build" button.
`here <https://readthedocs.io/projects/gammapy/>`__ and clicking the "Build" button.

This comment has been minimized.

@cdeil

cdeil May 13, 2016

Member

Same here, this should still point to .org.

@@ -237,7 +237,7 @@ def from_lower_and_upper_bounds(cls, lower, upper, unit=None):
"""
# np.append renders Quantities dimensionless
# http://astropy.readthedocs.org/en/latest/known_issues.html#quantity-issues
# http://docs.readthedocs.org/en/latest/known_issues.html#quantity-issues

This comment has been minimized.

@cdeil

This comment has been minimized.

@bsipocz

bsipocz May 13, 2016

Member

ohh, damn, sure. Thanks for catching these silly mistakes.

@bsipocz bsipocz force-pushed the bsipocz:changing_readthedocs_links branch from c465181 to 717ac27 May 13, 2016

@cdeil

This comment has been minimized.

Member

cdeil commented May 29, 2016

@bsipocz - Thank you!

Sorry, I had forgotten about this.
Merging this now.

@cdeil cdeil merged commit ba68986 into gammapy:master May 29, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment