Skip to content

Make links, data location and numerical description value un-translatable #140

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

Closed

Conversation

shaavan
Copy link
Contributor

@shaavan shaavan commented Aug 1, 2022

This is a follow-up to #124 and addresses this comment.

This PR removed the qsTr() tag for the description strings, which were links, disk locations, or numerical values, and hence wouldn't be translated from their source language.

@shaavan shaavan mentioned this pull request Aug 1, 2022
@shaavan shaavan changed the title qml: Make links, data location and numerical description value un-tra… Make links, data location and numerical description value un-tra… Aug 1, 2022
@shaavan shaavan changed the title Make links, data location and numerical description value un-tra… Make links, data location and numerical description value un-translatable Aug 3, 2022
@shaavan
Copy link
Contributor Author

shaavan commented Aug 12, 2022

Updated from 940a009 to 7ff6ff2 (pr140.01 -> pr140.02, diff)

Changes:

  • Rebased over main and resolved merge conflicts.

@shaavan shaavan force-pushed the 220802-onboarding-views-followup-1 branch from 940a009 to 7ff6ff2 Compare August 12, 2022 12:47
@jarolrod
Copy link
Member

should rebase over changes on master

@shaavan shaavan force-pushed the 220802-onboarding-views-followup-1 branch from 7ff6ff2 to 0b689ad Compare August 15, 2022 12:03
@shaavan
Copy link
Contributor Author

shaavan commented Aug 15, 2022

Updated from 7ff6ff2 to 0b689ad (pr140.02 -> pr140.03, diff)

Changes:

  • Rebased over main and made appropriate changes to account for updates in main.

@jarolrod
Copy link
Member

should rebase and incorporate with changes done on master since last push

@johnny9
Copy link
Collaborator

johnny9 commented Dec 17, 2022

concept ack

@jarolrod
Copy link
Member

Picked up in #217

@hebasto
Copy link
Member

hebasto commented Jan 16, 2023

Picked up in #217

Closing this one.

@hebasto hebasto closed this Jan 16, 2023
hebasto added a commit that referenced this pull request Jan 17, 2023
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.

4 participants