Release 1.56.0 #3664
Merged
Release 1.56.0 #3664
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Expecting this to fix the following error:
```
/home/circleci/project/gutenberg/node_modules/@react-native/polyfills/error-guard.js:14
type ErrorHandler = (error: mixed, isFatal: boolean) => void;
^^^^^^^^^^^^
SyntaxError: Unexpected identifier
at ScriptTransformer._transformAndBuildScript (node_modules/jest-runtime/node_modules/@jest/transform/build/ScriptTransformer.js:537:17)
at Object.<anonymous> (gutenberg/node_modules/react-native/jest/setup.js:434:6)
```
|
Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job! |
This was referenced Jun 25, 2021
|
LGTM! :) |
SiobhyB
approved these changes
Jun 25, 2021
geriux
pushed a commit
that referenced
this pull request
Jun 25, 2021
* Release 1.56.0 (#3664) * Update gutenberg reference * Update package-lock * Update gutenberg ref * Updated gutenberg ref * Updating gutenberg ref * Updated hermes executable to hermesc for building bundle * Updated bundle * Update gutenberg ref * Add changes to package-lock.json after running npm install * Disabled npm cache on CI * Update gutenberg ref * Add @react-native to transformIgnorePatterns in jest config Expecting this to fix the following error: ``` /home/circleci/project/gutenberg/node_modules/@react-native/polyfills/error-guard.js:14 type ErrorHandler = (error: mixed, isFatal: boolean) => void; ^^^^^^^^^^^^ SyntaxError: Unexpected identifier at ScriptTransformer._transformAndBuildScript (node_modules/jest-runtime/node_modules/@jest/transform/build/ScriptTransformer.js:537:17) at Object.<anonymous> (gutenberg/node_modules/react-native/jest/setup.js:434:6) ``` * Update gutenberg ref * Update gutenberg ref * updated gutenberg ref * Upgrade react-native related devDependencies * Update gutenberg ref * Fix check correctness diff error * Update gutenberg ref * Update gutenberg ref * Updated bundle files * Updated gutenberg ref * Updated gutenberg ref * Updated gutenberg ref * Adding cache-version file for re-running circle-ci * Update gutenberg ref * Updated gutenberg ref * Updated ref, removed react-native-mirror s3 from bridge gradle * updated bundle * Update podspecs by running generate-podspecs.sh script * Updating gutenberg ref * Update bundle * Update gutenberg ref * Revert "Disabled npm cache on CI" This reverts commit 3958c64. * Modify generate-podspecs.sh script to run FBReactNativeSpec codegen ahead of time * Update gutenberg ref * Update package-lock.json * Update package-lock.json * Add workaround for FBReactNativeSpec podspec in generate-poscpecs.sh script * Update commit hash of FBReactNativeSpec * Fix typo * Update commit hash of FBReactNativeSpec * Add missing external podspecs to generate-podspecs.sh script * Re-generate third-party-podspecs, remove react-native-linear-gradient podspec * Update commit hash of FBReactNativeSpec * Add exception for react-native-slider fork * Remove tag from react-native-slider podspec * Fix react-native-slider git source * Update gutenberg ref * Update bundles * Update gutenberg reference * Update release notes with fix for Gboard enter detection * Update gutenberg ref * Update jetpack ref * Update bundles * Update gutenberg ref * Update gutenberg ref * Re-run generate-podspecs.sh to update BVLinearGradient podspec * Update gutenberg ref * Update bundles * Shallow clone project for CI tasks Decrease CI task duration by reducing depth of git operations. - [CircleCI: Shallow Repository Cloning](https://support.circleci.com/hc/en-us/articles/360045096514-Shallow-Repository-Cloning) - [CircleCI: How do I modify the checkout step?](https://support.circleci.com/hc/en-us/articles/115015659247-How-do-I-modify-the-checkout-step-) - [Example: CircleCI Shallow Clone](https://discuss.circleci.com/t/shallow-clone-for-circleci-2-0-builds/20200) * Update Gutenberg ref * Updates gutenberg reference * Update Gutenberg submodule ref Fix component inspector error in React Native * Bump Gutenberg to latest trunk * Update gutenberg ref * Changes to package-lock.json after running npm install * Update bundles * Run ./bin/generate-podspecs.sh * Update gutenberg ref to merge commit * Update Gutenberg ref * Add a few improvements and checks to generate-podspecs.sh * Add a warning field at the bottom of all podspecs * Add a warning when the tag or commit field is missing in a podspec * Patch react-native-blur.podspec adding the missing tag * Better wording * Update bundles * Update FBReactNativeSpec ref * Indent patch messages * Patch react-native-blur.podspec again as 3.6.0 tag is missing * CI fetches commit reference directly The CI configuration is set to clone with a depth of 1. If the CI tasks then attempts to reference an older commit in the relevant branch, an error occurs. This might occur when a branch merge occurs between a CI tasks beginning and the point it attempt to fetch the HEAD of the branch. This change configures the shallow clone to fetch the specific commit reference, rather than the HEAD of the relevant branch. * Update gutenberg reference * Update gutenberg ref * Update gutenberg ref * Update gutenberg ref * Update gutenberg ref * Update bundles * Generate build * Moved item to the unreleased section of release notes * Update Gutenberg ref with merge commit * Bump Gutenberg hash * Update Gutenberg hash * Bump Gutenberg hash * Update Gutenberg hash * Update Gutenberg hash * Temporary Metro patch until RN upgrades Metro Issue message: "Error: EISDIR: illegal operation on a directory, read" Fix PR on Metro: facebook/metro#567 * Bump Gutenberg hash * Bump Gutenberg to the merged hash * Update Gutenberg ref * Update release notes * Fix incorrect block insertion point after blurring post title Selecting the title selection status within the same `useSelect` for reusable blocks caused the title selection status to become stale, due to the dependency array for the `useSelect` hook. The staleness caused the block insertion point to show up in the incorrect location after blurring the title text input. * Update gutenberg ref * Bump Gutenberg * Update RELEASE-NOTES.txt * Update Gutenberg hash which contains fix to Android artifact publishing * Bump Gutenberg * Add: Grid block, from block-experiments repo * Bump block-expeiments * Update the no_output_timeout to 15min * Revert the longer timeout. * Add block-experiments to eslintignore * Remove comment. * Fix prettier * Bump block-experiements * All Layout Grid controls * Bump block experiment fixes * Fix up the block setup code * Fix the availability * Update bundle * Simplify the layout grid block registration. * Update the bundle js files * Fix minor prettier issue. * Bump map since it changed * bump the latest version of Gutenberg * Update the bundle * Update gutenberg ref * Generate bundle * Bump Gutenberg * Update Gutenberg ref * Update gutenberg ref * Install latest packages * Install latest packages * Remove duplicative release note * Bump Gutenberg * Update extra node modules resolver Gutenberg mobile dependencies were resolved using the Jetpack path, so now we check first if the module exists in the Jetpack submodule before solving it with that path. * Remove email-validator dependency This dependency is already defined in Jetpack submodule and it's only used there. * Remove yarn.lock file * Remove yarn dependency * Clarify comment in extra node modules resolver * Update Gutenberg ref with merge commit * Bump Gutenberg * Release script: Update gb mobile version to 1.56.0 * Release script: Update gutenberg ref * Release script: Update bundle for: 1.56.0 * Update Release notes Co-authored-by: Cameron Voell <cameronvoell@gmail.com> Co-authored-by: Ceyhun Ozugur <ceyhunozugur@gmail.com> Co-authored-by: Amanda Riu <amanda.riu@automattic.com> Co-authored-by: David Calhoun <438664+dcalhoun@users.noreply.github.com> Co-authored-by: Carlos Garcia <fluiddot@gmail.com> Co-authored-by: Antonis Lilis <antonis.lilis@automattic.com> Co-authored-by: Paul Von Schrottky <paul.von.schrottky@automattic.com> Co-authored-by: Enej Bajgoric <enej.bajgoric@automattic.com> Co-authored-by: AmandaRiu <5810477+AmandaRiu@users.noreply.github.com> Co-authored-by: Stefanos Togkoulidis <stefanostogoulidis@gmail.com> Co-authored-by: Siobhan <siobhan@automattic.com> Co-authored-by: Oguz Kocer <oguz.kocer@automattic.com> Co-authored-by: Oguz Kocer <oguzkocer@users.noreply.github.com> Co-authored-by: Siobhan Bamber <SiobhyB@users.noreply.github.com> * Update Gutenberg ref Co-authored-by: Cameron Voell <cameronvoell@gmail.com> Co-authored-by: Ceyhun Ozugur <ceyhunozugur@gmail.com> Co-authored-by: Amanda Riu <amanda.riu@automattic.com> Co-authored-by: David Calhoun <438664+dcalhoun@users.noreply.github.com> Co-authored-by: Carlos Garcia <fluiddot@gmail.com> Co-authored-by: Antonis Lilis <antonis.lilis@automattic.com> Co-authored-by: Paul Von Schrottky <paul.von.schrottky@automattic.com> Co-authored-by: Enej Bajgoric <enej.bajgoric@automattic.com> Co-authored-by: AmandaRiu <5810477+AmandaRiu@users.noreply.github.com> Co-authored-by: Stefanos Togkoulidis <stefanostogoulidis@gmail.com> Co-authored-by: Siobhan <siobhan@automattic.com> Co-authored-by: Oguz Kocer <oguz.kocer@automattic.com> Co-authored-by: Oguz Kocer <oguzkocer@users.noreply.github.com> Co-authored-by: Siobhan Bamber <SiobhyB@users.noreply.github.com>
🟢 Writing Flow checklist (iOS)Build: 17.7.0.1 General
Rich Text Format
Splitting and merging
Undo / Redo - Test Cases
|
🟢 Unsupported Block Tests (iOS)Build: 17.7.0.1
|
🟢 New features/changes (iOS)Build: 17.7.0.1
|
🟢 Writing Flow checklist (Android)Build: 17.7.0.1 General
Rich Text Format
Splitting and merging
Undo / Redo - Test Cases
|
🟢 Unsupported Block Tests (Android)Build: 17.7.0.1
|
⚪ New features/changes (Android)Build: 17.7.0.1
|
🟢 iOS: Sanity Testing (Test Suite 2, Story Block 1)Build: 17.7.0.1 from Testflight DarkMode - 1
Group - 1
Spacer - 1
Buttons-1
Buttons-2
Buttons-3Button-6
Color Settings - 1
Image - 1
Story block - 1 |
Android Sanity Tests - Test Suite 1Build: 17.7-rc-1 ❓ I observed some issues while uploading videos (noted below in the Media & Text block tests) Gallery - 1
Gallery - 2
Gallery - 3
Columns - 1
Columns - 2
Columns - 3
MediaText - 1
MediaText - 2❓ Note: Videos thumbnails are not displaying in the device picker. Screenshot
Also, although not part of these tests, for a successful upload, the video URL seems to be incorrect, and the video does not play on the front-end. Steps
MultipleUpload - 1
Cover - 1
Editor Theme - 1
Social Icons - 1 |
|
Hey @mkevins 👋 thanks for testing!
By the URL being incorrect what do you mean? Was it a local URL? I wonder if the video got stuck processing 🤔. Also what kind of site were you using? |
Ah, I should have clarified. It was not a local (
I'm not sure, but I know it fully updated, since I can find the videos in my media library. This was on a .com site with a business plan. In the media library, I can see that the video URL is actually supposed to end with ".mp4" not "-mp4/", so maybe that's a clue. Also, the metadata listed there looks wrong though 🤔 . Fields for "FILE NAME" and "FILE TYPE" are blank, and "FILE SIZE" is "0.00 B".. so perhaps that is also a clue. |
7 tasks
Thanks for the information, I tried to upload a video using the Media&Text block in both free and premium site and it worked correctly, so I wonder if it was something specific for that video file? |
|
I've just added the |
@geriux 👋 😃 Out of curiousity, when you look at the final markup for the video after it uploaded successfully, does it end with ".mp4", or "-mp4/" in your case? Or something else? Btw, I encountered this issue on a Business site.
I'm not sure, but I don't think so? I used a 10 second clip from Big Buck Bunny, and the video is actually available in the media library afterwards, so I know it uploaded. 🤔 |
For me, it ends with
Interesting, when I tested I used the camera to record a 4 seconds video. |
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Release for Gutenberg Mobile 1.56.0
Related PRs
Gutenberg: Release 1.56.0 #3664
WPAndroid: Integrate gutenberg-mobile release 1.56.0 WordPress-Android#14936
WPiOS: Integrate gutenberg-mobile release 1.56.0 WordPress-iOS#16752
Aztec-iOS: https://github.com/wordpress-mobile/AztecEditor-iOS/pull/
Aztec-Android: https://github.com/wordpress-mobile/AztecEditor-Android/pull
Extra PRs that Landed After the Release Was Cut
No extra PRs yet. 🎉
Changes
Test plan
Release Submission Checklist
RELEASE-NOTES.txtis updated with all the changes that made it to the release.gutenberg/packages/react-native-editor/CHANGELOG.mdis updated with all the changes that made it to the release.