-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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 and 18.9.0.1 and 18.9.0.2 into develop
#17695
Conversation
…tton Follow Conversation: Fix cases where the follow button is missing
…storation-crash Reader Comments: Fix crash when the app is backgrounded
…ng is still WIP We are currently working on removing `update-translations.rb` + `fix-translations` + `extract-framework-translations.swift` scripts from the repo and migrating their logic into the release-toolkit, but that work is still in progress. The role of the (misleadingly named) `extract-frameworks-translations.swift` script was to redistribute the keys from the `*.lproj/Localizable.strings` downloaded from GlotPress into the `.strings` files of the Widgets and ShareExtension targets, back when those targets used their own `.strings` file. This is no longer the case since #17630 and #17636 landed – as Widgets and ShareExtension's code now reference the app's `.strings` file directly – which means we don't need to run the logic from `extract-frameworks-translations.swift` anymore, and in fact if we did, it would probably crash now that the `Base.lproj/Localizable.strings` files in the Widgets and ShareExtension subfolders (which that script references) don't exist anymore.
…king Comments: Track comment editor opened.
That's because of the `develop` to `trunk` rename.
Got conflicts in `Gemfile.lock` and `fastlane/Pluginfile` because both branches changed the release-toolkit version. I resolved by using the version on `develop`.
You can trigger an installable build for these changes by visiting CircleCI here. |
develop
develop
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.
- 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
You can trigger optional UI/connected tests for these changes by visiting CircleCI here. |
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.
That's a lot of changes 😅
Includes:
From 18.9.0.1
From 18.9.0.2
Got conflicts in
Gemfile.lock
andfastlane/Pluginfile
because both branches changed the release toolkit version. I resolved by using the version ondevelop
.