Skip to content

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

Merged
merged 2 commits into from
Feb 6, 2025

Conversation

partev
Copy link
Contributor

@partev partev commented Apr 12, 2024

fix URL redirect http -> https

Brief description of what is fixed or changed

Copy link

netlify bot commented Apr 12, 2024

Deploy Preview for numpy-org ready!

Name Link
🔨 Latest commit 9fb24e6
🔍 Latest deploy log https://app.netlify.com/sites/numpy-org/deploys/67a462a5deb2470008b2f98c
😎 Deploy Preview https://deploy-preview-744--numpy-org.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

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
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@rgommers @steppi It looks like this isn't in content/pt/tabcontents.yaml or content/ja/tabcontents.yaml. Is there a plan or mechanism to keep the translations up-to-date?

Copy link
Member

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?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@steppi Gentle ping.

Copy link
Contributor

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

url: http://qutip.org
label: QuTiP

- 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.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you, Albert!

@stefanv stefanv merged commit 83ee56e into numpy:main Feb 6, 2025
4 checks passed
@stefanv
Copy link
Contributor

stefanv commented Feb 6, 2025

Thanks @partev!

@partev partev deleted the patch-1 branch February 6, 2025 13:20
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.

5 participants