Update urls #2849
4 participants
jwhitlock
commented on an outdated diff
| @@ -11,7 +11,7 @@ Coding conventions | ||
| We follow most of the practices as detailed in the `Mozilla webdev | ||
| bootcamp guide | ||
| -<http://mozweb.readthedocs.org/en/latest/coding.html>`_. | ||
| +<https://mozweb.readthedocs.io/en/latest/coding.html>`_. |
|
This is a 404. I think the URL should be https://mozweb.readthedocs.io/en/latest/guide/development_process.html
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
jwhitlock
commented on an outdated diff
| @@ -110,7 +110,7 @@ These settings explained: | ||
| There are a few other settings you can set in your | ||
| ``kitsune/settings_local.py`` file that override ElasticUtils defaults. See | ||
| `the ElasticUtils docs | ||
| -<http://elasticutils.readthedocs.org/en/latest/installation.html#configure>`_ | ||
| +<https://elasticutils.readthedocs.io/en/latest/installation.html#configure>`_ |
|
Link is OK, but I think https://elasticutils.readthedocs.io/en/latest/django.html#configuration is a better target.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
jwhitlock
commented on an outdated diff
| @@ -63,7 +63,7 @@ Git resources and tools | ||
| ----------------------- | ||
| See `Webdev bootcamp guide | ||
| -<http://mozweb.readthedocs.org/en/latest/git.html#git-and-github>`_ | ||
| +<https://mozweb.readthedocs.io/en/latest/git.html#git-and-github>`_ |
|
This is a 404. https://mozweb.readthedocs.io/en/latest/reference/git_github.html looks correct.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
|
I appreciate you finding correct urls. I lazily just did a couple of search-and-replaces.
I wonder if the incredible wrongness of some of these urls suggests the documentation in those areas is also horribly wrong.
^^^ That updates the urls you pointed out as being wrong.
+r. I don't have commit access, and don't want it, so someone else needs to merge.
r+ from my end. @willkg You may have commit access or @glogiotatidis can merge
r?