Skip to content
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

Remove all readthedocs pages and setup redirects to docs.cider.mx #2734

Merged
merged 2 commits into from
Oct 21, 2019

Conversation

danielcompton
Copy link
Contributor

@danielcompton danielcompton commented Oct 20, 2019

Once this is merged, we will need to:

  • rebuild latest and stable. This should delete all existing
    HTML sources and cause every request to hit the RTD servers. Then those
    requests will get a 302 redirect to their new counterpart on docs.cider.mx.
  • request RTD staff to change our 302 temporary redirects to 301
    redirects. This seems like a manual process

    Users can request a 301 redirect via our support queue and we just need to make a change on those redirects after they set them up and are happy.

Fixes #2695.


Before submitting the PR make sure the following things have been done (and denote this
by checking the relevant checkboxes):

  • The commits are consistent with our contribution guidelines
  • N/A You've added tests (if possible) to cover your change(s)
  • N/A All tests are passing (make test)
  • N/A All code passes the linter (make lint) which is based on elisp-lint and includes
  • N/A You've updated the changelog (if adding/changing user-visible functionality)
  • N/A You've updated the user manual (if adding/changing user-visible functionality)

Because:
- We have setup page redirects in the RTD backend which will redirect
all versions of the CIDER site to the new docs.cider.mx docs site.
- Once we delete the generated pages, all requests will hit the RTD
backend and get 302 (temporary) redirected to the new docs.cider.mx site.
@bbatsov bbatsov merged commit 3e7b8bc into clojure-emacs:master Oct 21, 2019
@danielcompton
Copy link
Contributor Author

I've rebuilt the latest and stable docs. I had to quickly create a tag 0.23.1 to rebuild stable, which I've now removed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Apply some SEO to cider.mx
2 participants