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

Release 1.104.0 #6204

Merged
merged 11 commits into from
Sep 25, 2023
Merged

Release 1.104.0 #6204

merged 11 commits into from
Sep 25, 2023

Conversation

fluiddot
Copy link
Contributor

@fluiddot fluiddot commented Sep 14, 2023

Release for Gutenberg Mobile 1.104.0

Related PRs

Extra PRs that Landed After the Release Was Cut

No extra PRs yet. 🎉

Changes

RichText - Update logic for the placeholder text color

feat: Insert from URL leverages a bottom sheet

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

  • Verify Items from test plan have been completed
  • Check if RELEASE-NOTES.txt is updated with all the changes that made it to the release. Replace Unreleased section with the release version and create a new Unreleased section.
  • Check if 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.
  • Bundle package of the release is updated.

@@ -348,9 +348,6 @@ translators: %s: Select control option value e.g: "Auto, 25%". -->
<string name="gutenberg_native_slash_inserter_results" tools:ignore="UnusedResources">Slash inserter results</string>
<string name="gutenberg_native_some_blocks_have_additional_settings_tap_the_settings_icon_on_the" tools:ignore="UnusedResources">Some blocks have additional settings. Tap the settings icon on the bottom right of the block to view more options.</string>
<string name="gutenberg_native_start_writing" tools:ignore="UnusedResources">Start writing…</string>
<string name="gutenberg_native_take_a_photo" tools:ignore="UnusedResources">Take a Photo</string>
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 string seems to be only used in the native version, so these changes should be reverted. Not sure why the script in charge of updating the localizations has removed them, I'd need to investigate it further 😞 .

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fixed in 1b95e55.

# Conflicts:
#	bundle/ios/App.js
#	bundle/ios/App.js.map
#	bundle/ios/App.text.js
#	bundle/ios/App.text.js.map
#	gutenberg
#	ios-xcframework/Podfile.lock
#	package-lock.json
#	package.json
@@ -188,7 +188,7 @@
"eslint-plugin-jsdoc": "^46.4.6",
"eslint-plugin-jsx-a11y": "^6.5.1",
"eslint-plugin-playwright": "^0.15.3",
"eslint-plugin-prettier": "^3.3.0",
"eslint-plugin-prettier": "^5.0.0",
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 change is related to WordPress/gutenberg#54539.

@fluiddot
Copy link
Contributor Author

The failure in Lint and Unit Tests will be solved in #6219.

Error:

TypeError: Error while loading rule 'prettier/prettier': context.getPhysicalFilename is not a function
Occurred while linting /workdir/__device-tests__/gutenberg-editor-functionality-test-1-visual.test.js
    at Object.create (/workdir/node_modules/eslint-plugin-prettier/eslint-plugin-prettier.js:140:40)
    at createRuleListeners (/workdir/node_modules/eslint/lib/linter/linter.js:746:21)
    at /workdir/node_modules/eslint/lib/linter/linter.js:916:31
    at Array.forEach (<anonymous>)
    at runRules (/workdir/node_modules/eslint/lib/linter/linter.js:861:34)
    at Linter._verifyWithoutProcessors (/workdir/node_modules/eslint/lib/linter/linter.js:1157:31)
    at Linter._verifyWithConfigArray (/workdir/node_modules/eslint/lib/linter/linter.js:1255:21)
    at Linter.verify (/workdir/node_modules/eslint/lib/linter/linter.js:1210:25)
    at Linter.verifyAndFix (/workdir/node_modules/eslint/lib/linter/linter.js:1400:29)
    at verifyText (/workdir/node_modules/eslint/lib/cli-engine/cli-engine.js:230:48)

# Conflicts:
#	gutenberg
#	ios-xcframework/Podfile.lock
#	package-lock.json
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants