-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
chore: Update GitHub pages to redirect to readthedocs #12362
chore: Update GitHub pages to redirect to readthedocs #12362
Conversation
so, is it through these html files that people will mainly be getting to the documentation now? or only in the case that they're looking for a specific version of the documentation, and if not through a redirect from https://argoproj.github.io/argo-workflows? |
Actually, I guess the redirect takes one to "stable" - so that implies that if someone navigated to one of these files within a given Github release tag, they would not be redirected to readthedocs for that version, right? |
@juliev0 these redirects will ensure people get to the new docs location and to the files they had old links to. Great catch on |
So, it looks like we no longer publish to https://argoproj.github.io/argo-workflows? Will there be any redirect from there or no? |
Oh, never mind. These index.html pages are still being published to https://argoproj.github.io/argo-workflows, aren't they? |
@juliev0 yes, they'd replace the github pages and still be published. |
448f5b3
to
809c1f4
Compare
Let me know if this is an unreasonable request, but would it make any sense to attach the script you used to make the change as something that could be reviewed instead of trying to review all of the docs? |
809c1f4
to
b79ba30
Compare
@juliev0 https://gist.github.com/jmeridth/94cd15d2b11d444fdf91913aa59f34ab (please don't judge 😉, was written in under an hour) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thanks for modifying these all! Looks like you got nearly everything; I was checking for subdirs in particular and you got all the ones I checked.
I left a few comments in-line for some improvements and/or missing pieces
b79ba30
to
6aec673
Compare
@agilgur5 as these updates are generated from my script (updated with your latest suggestions), I am running locally and pushing. Don't mean to not give credit for the contributions, suggestions. You cool with me adding you as a co-author? |
nbd, I don't really mind either way. thanks for asking though! |
Really appreciate your doing that, even though I know it's an unusual request! |
e17fcd7
to
e10cd7e
Compare
This prevents overwriting of the redirect pages we will have in gh-pages branch now due to argoproj#12362 Signed-off-by: jmeridth <[email protected]>
e10cd7e
to
40f106f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Two small corrections, otherwise LGTM
ec52a01
to
1173728
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for automating all the redirects and iterating on this!
One tiny thing I noticed, that I don't think we can really do much about, is that URL anchor will be lost in the redirect. I.e. if you had But adding the anchor requires front-end JS or back-end logic. We have no back-end in GH Pages and the EDIT: interestingly, the |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Did you mean to mark this as ready for review?
Yes for review but #12360 has to merge first. I'm ready to move on both. |
1173728
to
853b544
Compare
Don't know why this closed. @terrytangyuan are you able to reopen? |
Did you recreate the branch with the same name and force pushed? That might be why. Not 100% sure though. |
I cannot re-open it somehow |
Yeah, had to. Let me see if I can do it the other way. One sec. |
New PR it is then. Dang it. Sorry. |
Previously was argoproj#12212 and argoproj#12362 Relates to argoproj#12360 ### Motivation Change each GitHub page to utilize meta tag to redirect to new location on readthedocs Since we don't host our previous documentation we can't do a real HTTP redirect. Meta tag is the next best thing https://www.w3.org/TR/WCAG20-TECHS/H76.html ### Modifications - [x] setup redirects to [readthedocs](https://argo-workflows.readthedocs.io/en/latest/) + previus page suffix (once argoproj#12360 merges and builds) [Python script used to update the files](https://gist.github.com/jmeridth/94cd15d2b11d444fdf91913aa59f34ab) Signed-off-by: jmeridth <[email protected]>
Previously was argoproj#12212 and argoproj#12362 Relates to argoproj#12360 ### Motivation Change each GitHub page to utilize meta tag to redirect to new location on readthedocs Since we don't host our previous documentation we can't do a real HTTP redirect. Meta tag is the next best thing https://www.w3.org/TR/WCAG20-TECHS/H76.html ### Modifications - [x] setup redirects to [readthedocs](https://argo-workflows.readthedocs.io/en/latest/) + previus page suffix (once argoproj#12360 merges and builds) [Python script used to update the files](https://gist.github.com/jmeridth/94cd15d2b11d444fdf91913aa59f34ab) Signed-off-by: jmeridth <[email protected]>
Replaced by #12443 |
Previously was #12212
Relates to #12360
Motivation
Change each GitHub page to utilize meta tag to redirect to new location on readthedocs
Since we don't host our previous documentation we can't do a real HTTP redirect. Meta tag is the next best thing
https://www.w3.org/TR/WCAG20-TECHS/H76.html
Modifications
Python script used to update the files