-
-
Notifications
You must be signed in to change notification settings - Fork 120
Update tabcontents.yaml fix URL redirect http -> https #744
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
Conversation
✅ Deploy Preview for numpy-org ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
@@ -89,7 +89,7 @@ params: | |||
alttext: A computer chip. | |||
img: /images/content_images/sc_dom_img/quantum_computing.svg | |||
links: | |||
- url: http://qutip.org | |||
- url: https://qutip.org |
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.
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.
@steppi could you please have a look at this?
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.
@steppi Gentle ping.
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.
Sorry. I'm not sure how I missed the pings back in April. This url has made it into content/ja/tabcontents.yaml
and content/pt/tabcontents/yaml
now
numpy.org/content/ja/tabcontents.yaml
Lines 102 to 103 in f9a497c
url: http://qutip.org | |
label: QuTiP |
numpy.org/content/pt/tabcontents.yaml
Line 87 in f9a497c
- url: http://qutip.org |
The mechanism for keeping translations up to date is that updates to the source content propagate to Crowdin through the GitHub integration and translators get notified that there is new or updated content to translate. The updates made in Crowdin are pushed to the running branch l10n_main
and we have a GitHub action which essentially cherry picks commits from this branch to create PRs for particular languages.
Process-wise, it would be fine either to just update the url in content/en/tabcontents.yaml
and let the other updates be made by translators through Crowdin, or to make the change for all of the translated urls directly in GitHub. Originally, we had Crowdin configured in such a way that made it difficult to resolve conflicts when changes are made outside of Crowdin but that isn't an issue any more.
In this particular case, I think it's better the change http
-> https
directly in GitHub for the different languages, since this isn't really something that should require translator attention. As an aside, I want to figure out how to hide things like urls and other strings which should not be translated from the translators.
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.
Thank you, Albert!
fix URL redirect http -> https
Thanks @partev! |
fix URL redirect http -> https
Brief description of what is fixed or changed