-
Notifications
You must be signed in to change notification settings - Fork 58
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
Release 1.105.0 #6243
Release 1.105.0 #6243
Conversation
bundle/android/strings.xml
Outdated
<string name="gutenberg_native_take_a_photo" tools:ignore="UnusedResources">Take a Photo</string> | ||
<string name="gutenberg_native_take_a_photo_or_video" tools:ignore="UnusedResources">Take a Photo or Video</string> | ||
<string name="gutenberg_native_take_a_video" tools:ignore="UnusedResources">Take a Video</string> |
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.
#6204 (review), the i18n script is marking these strings as used by the web, so it considers that we don't need to translate them as they would be part of the Gutenberg GlotPress project. However, I checked that these strings are only referenced by native files, hence they won't be translated unless we add them here.
I presume that the fact that these strings are in an extension different from .native.js
is making the i18n script consider them as used on the web. Looks like the constants associated with the localized strings are only used in an integration test, not sure if this might confuse the i18n script regarding whether they are used in web or not. In any case, as a workaround, we could simply move the test-only constants to a separate file under the test
folder. I'll open a separate PR to address this and prevent this issue in future releases.
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.
Fixed in 5ac4d8b.
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.
Next releases won't encounter this issue once we merge WordPress/gutenberg#54928.
gutenberg
Outdated
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.
The Gutenberg reference is not pointing to the latest commit from WordPress/gutenberg#54898. However, I understand that the latest changes in the PR aren't incorporating anything critical that we'd like to check in the CI test jobs.
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.
LGTM 🎊 !
# Conflicts: # gutenberg
<string name="gutenberg_native_take_a_photo" tools:ignore="UnusedResources">Take a Photo</string> | ||
<string name="gutenberg_native_take_a_photo_or_video" tools:ignore="UnusedResources">Take a Photo or Video</string> |
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.
Curiously enough, this string is not used anywhere in the Gutenberg codebase 🤔 .
🟢 Writing Flow (iOS)Build: 23.4.0.0 (TestFlight) General
Rich Text Format
Splitting and merging |
🟢 Unsupported Block Tests (iOS)Build: 23.4.0.0 (TestFlight)
|
🟢 Functionality Test Suite 1 (iOS)Build: 23.4.0.0 (TestFlight) Gallery - 1
Gallery - 2
MediaText - 1
MediaText - 2
MultipleUpload - 1
Cover - 1
Editor Theme - 1 |
🟢 New Changes (iOS)Build: 23.4.0.0 (TestFlight) |
🟢 Writing Flow (Android)Build: 23.4-rc-1 General
Rich Text Format
Splitting and merging |
🟢 Unsupported Block Tests (Android)Build: 23.4-rc-1
|
🟢 Functionality Test Suite 5 (Android)Build: 23.4-rc-1 Video block - 1
Video block - 2
File block - 1
File block - 2
File block - 3
Audio block - 1
Audio block - 2
Audio block - 3
Audio block - 4
Synced patterns - 1 Synced patterns - 2 Embed block - 1
Embed block - 2
|
🟢 New Changes (Android)Build: 23.4-rc-1 |
Release for Gutenberg Mobile 1.105.0
Related PRs
Extra PRs that Landed After the Release Was Cut
No extra PRs yet. 🎉
Changes
Change 1: Prevent crash from invalid media URLs
Change 2: Limit inner blocks nesting depth to avoid call stack size exceeded crash
Change 3: Fallback to Twitter provider when embedding X URLs
Test plan
Once the installable builds of the main apps are ready, perform a quick smoke test of the editor on both iOS and Android to verify it launches without crashing. We will perform additional testing after the main apps cut their releases.
Release Submission Checklist
RELEASE-NOTES.txt
is updated with all the changes that made it to the release. ReplaceUnreleased
section with the release version and create a newUnreleased
section.gutenberg/packages/react-native-editor/CHANGELOG.md
is updated with all the changes that made it to the release. Replace## Unreleased
with the release version and create a new## Unreleased
.