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

Merge 18.7.0.1 (beta 2) into develop #17562

Merged
merged 12 commits into from
Nov 25, 2021
Merged

Merge 18.7.0.1 (beta 2) into develop #17562

merged 12 commits into from
Nov 25, 2021

Conversation

mokagio
Copy link
Contributor

@mokagio mokagio commented Nov 25, 2021

I made this to ship a build with the most up-to-date translations we have to help smoke testing them in the app.

Includes:

Also notice the new subtitles for Jetpack, and that quite a few of them go over the 30 characters limit 😞

The request for the translator is visible in GlotPress, albeit not "in your face". It would be nice if we could have some kind of annotation in the .po file that GlotPress can read and use to enforce a characters threshold.

Screen Shot 2021-11-25 at 2 47 53 pm

@peril-wordpress-mobile
Copy link

You can trigger an installable build for these changes by visiting CircleCI here.

ووردبريس أسرع وأكثر أمانًا
Copy link
Contributor Author

Choose a reason for hiding this comment

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

26 characters ✅

WordPress – schneller und sicherer
Copy link
Contributor Author

Choose a reason for hiding this comment

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

34 characters ❌

WordPress más rápido y seguro
Copy link
Contributor Author

Choose a reason for hiding this comment

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

29 characters ✅

WordPress plus sûr, plus rapide
Copy link
Contributor Author

Choose a reason for hiding this comment

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

31 characters ❌

Faster, safer WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This obviously hasn't been translated yet

WordPress lebih cepat dan aman
Copy link
Contributor Author

Choose a reason for hiding this comment

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

30 characters ✅, just 😅

WordPress più veloce e più sicuro
Copy link
Contributor Author

Choose a reason for hiding this comment

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

33 characters ❌

I'll update in GlotPress to "WordPress più veloce e sicuro" which means the same in Italian and is 29 characters.

より速く安全な WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

17 characters ✅

That is if App Store Connect does something like "...".length, which would make sense to me given that the 30 characters limitation is not a matter of sentence complexity but of screen real estate. I'm not actually sure if Japanese strings kanji are counted in the same way as we do with the letters in a Roman language.

더 빠르고 더 안전한 워드프레스
Copy link
Contributor Author

Choose a reason for hiding this comment

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

17 characters ✅

That is if App Store Connect does something like "...".length, which would make sense to me given that the 30 characters limitation is not a matter of sentence complexity but of screen real estate. I'm not actually sure if Korean characters are counted in the same way as we do with the letters in a Roman language.

Sneller, veiliger WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

27 characters ✅

O WordPress mais ágil e seguro
Copy link
Contributor Author

Choose a reason for hiding this comment

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

30 characters ✅

Более быстрый и безопасный WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

36 characters ❌

Snabbare, säkrare WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

27 characters ✅

Daha hızlı, daha güvenli bir WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

38 characters ❌

更快速、更安全的 WordPress
Copy link
Contributor Author

Choose a reason for hiding this comment

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

18 characters ✅

That is if App Store Connect does something like "...".length, which would make sense to me given that the 30 characters limitation is not a matter of sentence complexity but of screen real estate. I'm not actually sure if Chinese characters are counted in the same way as we do with the letters in a Roman language.

更快速安全的 WordPress 體驗
Copy link
Contributor Author

Choose a reason for hiding this comment

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

19 characters ✅

That is if App Store Connect does something like "...".length, which would make sense to me given that the 30 characters limitation is not a matter of sentence complexity but of screen real estate. I'm not actually sure if Chinese characters are counted in the same way as we do with the letters in a Roman language.

Copy link
Contributor

Choose a reason for hiding this comment

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

Can confirm that's how they're counted 🙂

image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Oh, thank you for checking ❤️ TIL

Copy link
Contributor Author

@mokagio mokagio left a comment

Choose a reason for hiding this comment

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

  • Updates to the localization files (.strings and Fastlane metadata) automatically pulled for the new translations that have already been approved in GlotPress
  • Version update in .xcconfig
  • Diffs from the PRs that made it into this beta

@mokagio mokagio requested a review from a team November 25, 2021 03:48
@mokagio mokagio added this to the 18.7 ❄️ milestone Nov 25, 2021
@peril-wordpress-mobile
Copy link

Messages
📖 This PR has the 'Releases' label: some checks will be skipped.

Generated by 🚫 dangerJS

@mokagio mokagio enabled auto-merge November 25, 2021 03:49
@AliSoftware AliSoftware changed the title Merge 18.7.0.2 into develop Merge 18.7.0.1 (beta 2) into develop Nov 25, 2021
Copy link
Contributor

@AliSoftware AliSoftware left a comment

Choose a reason for hiding this comment

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

Looking good.

Approving despite some subtitles being >30 because you already made a request in +i18n to ask them to be fixed by translators (ref: pxLjZ-6GO-p2) so best scenario they're gonna be fixed by the time you submit on Friday, and worst case you'll just skip updating them this time around and hope they get fixed for the next.

PS: Edited the PR's title which was mentioning 18.7.0.2 while this is 18.7.0.1 (beta 2, but zero-based versioning, yadda yadda)

@mokagio mokagio merged commit 271409a into develop Nov 25, 2021
@mokagio
Copy link
Contributor Author

mokagio commented Nov 25, 2021

PS: Edited the PR's title which was mentioning 18.7.0.2 while this is 18.7.0.1 (beta 2, but zero-based versioning, yadda yadda)

Lol. Yeah I should have actually called this beta 1 (with code freeze being beta 0). I think in my mind doing the first beta on the Thursday before finalization was just too weird and my subconscious just assumed this was another one 🤣

Thanks for fixing it.

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

Successfully merging this pull request may close these issues.

4 participants