Skip to content

Conversation

@enejb
Copy link
Contributor

@enejb enejb commented Mar 1, 2021

Release for Gutenberg Mobile 1.48.0

Related PRs

Extra PRs that Landed After the Release Was Cut

None

Changes

Test plan

  • Use the main WP apps to test the changes above.
  • Smoke test the main WP apps for general writing flow.

Notes

The following PR change was reverted on this branch:

Release Submission Checklist

  • Approve and run optional Android and iOS UI tests
  • Check if RELEASE-NOTES.txt and gutenberg/packages/react-native-editor/CHANGELOG.md are updated with all the changes that made it to the release.
  • Bundle package of the release is updated.

Paul Von Schrottky and others added 30 commits February 15, 2021 21:53
…dth-attribute

Ignore column width attribute when empty
* Enable default keyboard as default on link picker text fields

* Update Lock file

* Update Gutenberg for merged PR

* Revert to develop package-lock.json
* Update ref to branch: rnmobile/uderline-linked-selected-text

* Update Gutenberg Ref

Co-authored-by: Chip Snyder <[email protected]>
…o_develop

Merge release 1.47.0 to develop
…-initial-html

Add audio block to initial html
…ence-to-master

Update Gutenberg reference to point merge commit in master
@chipsnyder
Copy link
Contributor

RE: #3205 (comment)

It is a theme problem. In the theme_supports
the editor-color-palette is #fcfcfc (white) for 4/5 of the colors and the gradients are all a variation of:
linear-gradient(to bottom right, #fcfcfc 49.9%, #fcfcfc 50%) or white to white.

cc @geriux

@geriux
Copy link
Contributor

geriux commented Mar 4, 2021

RE: #3205 (comment)

It is a theme problem. In the theme_supports
the editor-color-palette is #fcfcfc (white) for 4/5 of the colors and the gradients are all a variation of:
linear-gradient(to bottom right, #fcfcfc 49.9%, #fcfcfc 50%) or white to white.

cc @geriux

Thanks for the confirmation @chipsnyder!

@geriux
Copy link
Contributor

geriux commented Mar 4, 2021

Android Writing Flow checklist (round 2)

Device: Redmi Note 8T
OS: Android 10
Build: 98127

General

  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@geriux
Copy link
Contributor

geriux commented Mar 4, 2021

iOS Writing Flow checklist

Device: iPhone 12 Pro
OS: iOS 14.4
Build: 43343

Writing Flow checklist

General

  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@geriux
Copy link
Contributor

geriux commented Mar 4, 2021

Update

🟢 Writing flow tests on Android passed
🟢 Writing flow tests on iOS passed
🟢 Test Suite 1 (Android) sanity test passed
🟡 Test Suite -1 (iOS) (pending reply from this comment)
🟡 Story block - 1 (Android) (pending reply from this comment)
🟢 Test Suite -2 (iOS) sanity test passed
🟢 Test Suite 4 (iOS) sanity test passed

@mchowning
Copy link
Contributor

mchowning commented Mar 4, 2021

🟡 TC005 - Story block - Close/Re-open post with an ongoing Story upload

I'm getting crashes on the Android activity when leaving with an ongoing upload.
This seems to be related to the deliberate decision here but this only crashes on debug builds. @mchowning I was getting the crash consistently if that helps to track down the original issue.

@chipsnyder @mchowning is this a blocker for the release? Is it the same behavior we have now on production or is this a regression that we should check?

I don't think this is a release blocker. Worst case scenario, autosave isn't working, but the changes will still be saved on exit (that's what happens instead of the crash on release builds).

UPDATE: Let's test this on a release build to make sure everything looks ok there.

@enejb
Copy link
Contributor Author

enejb commented Mar 4, 2021

@geriux
🟡 Test Suite -1 (iOS) This seems to be present currently.
I am not seeing the left and right arrows on the views that see
image

But this doesn't seem to be a regression since it is already present in the current app. Since the user is still able to switch columns around when they turn the app vertically I would say we can fix this in a new release.

I created a new issue for it WordPress/gutenberg#29570

@enejb
Copy link
Contributor Author

enejb commented Mar 4, 2021

Test Suite 3

Gallery - 2

  • Gallery block - Insert image from device (failing) - TC006
  • Gallery block - Insert image from device (cancel) - TC009

Gallery - 3

  • Gallery block - Add Gallery Caption - TC003
  • Gallery block - Add Gallery Image Caption - TC004
  • Gallery block - Try adding same images from WP Media library and moving the images around - TC010

Gallery - 4

  • Gallery block - Settings: Link to - TC012
  • Gallery block - Settings: Column number - TC013
  • Gallery block - Settings: Crop images - TC014

MediaText - 3

  • Media Text block - Insert video from device (failing) - TC001-v
  • Media Text block - Close/Re-open post with an ongoing video upload - TC003-v

MediaText - 4

  • Media Text block - Media & Text alignment - TC006
  • Media Text block - Vertical alignment - TC007

Cover - 2

🔴 Cover - Add Video Background from WordPress Media library - TC005
When I add a video background and then preview the post I end up with a broken cover block.

  • Cover - Replace media - TC006
  • Cover - Gradient background is rendered properly - TC007
  • Cover - Gradient overlay is rendered properly - TC008

Social Icons - 2

  • The link sheet is opened automatically when adding new icon - TC004
  • Ghost placeholder is visible when no icon is active - TC005
  • Social icon forwarding to the link - TC006

Image - 2

  • Image block - Switch to classic editor with an image block in page - TC006
  • Image block - Border style setting - TC007
  • Image block - Image size setting - TC008
  • Image block - Link to setting - TC009

Story block - 1

🔴 Story block - Verify is available in Block Picker - TC001 - I don't see the story block in the list of blocks.

  • Story block - Verify is not available in Block Picker- TC002
  • Story block - Add media to empty Story block - TC003
  • Story block - Add media to non-empty Story block - TC004
  • Story block - Close/Re-open post with an ongoing Story upload - TC005

@geriux
Copy link
Contributor

geriux commented Mar 5, 2021

@geriux
🟡 Test Suite -1 (iOS) This seems to be present currently.
I am not seeing the left and right arrows on the views that see

I see it now @enejb, since there's no much space for the arrows controls we put them under the three dots button:

For larger screens like an iPad with a Columns block with full-width alignment you'll be able to see them:

@geriux
Copy link
Contributor

geriux commented Mar 5, 2021

🔴 Story block - Verify is available in Block Picker - TC001 - I don't see the story block in the list of blocks.

@enejb can you confirm you were testing on iOS? If so, this block is still not out for production.

@geriux
Copy link
Contributor

geriux commented Mar 5, 2021

🔴 Cover - Add Video Background from WordPress Media library - TC005
When I add a video background and then preview the post I end up with a broken cover block.

@enejb This is a premium block for videos, were you testing on a paid plan?

@antonis
Copy link

antonis commented Mar 5, 2021

Writing Flow checklist

Android build from wordpress-mobile/WordPress-Android#14166
Device: Pixel 2 XL (Android 11)

General

  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@fluiddot
Copy link
Contributor

fluiddot commented Mar 5, 2021

iOS Writing Flow checklist

Device: iPhone 11
OS: iOS 14.2
Build: 43443

Writing Flow checklist

General

  • TC001 - Block insertion flow
  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components

Rich Text Format

  • TC001 - Bold, Italic, strikethrough buttons
  • TC002 - Alignment buttons
  • TC003 - Alignment Split
  • TC004 - Link button works without selection
  • TC005 - Link button works with a selected word
  • TC006 - Adding a link from a copied URL
  • TC007 - Test format detection under the cursor
  • TC008 - Test formatting doesn't remove leading or trailing whitespace
Splitting and merging
  • TC001 - Merge after writing
  • TC002 - Merge after selection
  • TC003 - Merge after deleting text
  • TC004 - Merge after deleting all
  • TC005 - Merge multiple blocks
  • TC006 - Splitting/merge list block

Undo / Redo - Test Cases

  • TC001 - Undo/redo block actions
  • TC002 - Undo/redo text
  • TC003 - Undo/redo text format

@enejb
Copy link
Contributor Author

enejb commented Mar 5, 2021

@geriux

can you confirm you were testing on iOS? If so, this block is still not out for production.
Yes I was testing on iOS. So I guess this is expected then.

This is a premium block for videos, were you testing on a paid plan?
Yes I was using on a simple site. I think this might not be a blocker for merging. Mostly since it doesn't
But it would be good to see if can get this fixed in the near future.

@geriux
Copy link
Contributor

geriux commented Mar 5, 2021

🟢 All issues have been solved, this is ready to :shipit:

Copy link
Contributor

@geriux geriux left a comment

Choose a reason for hiding this comment

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

LGTM! Thank you to everyone involve in testing this release!

@enejb enejb merged commit 12d7b2c into trunk Mar 5, 2021
@enejb enejb deleted the release/1.48.0 branch March 5, 2021 16:37
@fluiddot fluiddot restored the release/1.48.0 branch March 9, 2021 15:22
@fluiddot fluiddot deleted the release/1.48.0 branch March 9, 2021 15:50
@fluiddot
Copy link
Contributor

🔴 Story block - Verify is available in Block Picker - TC001 - I don't see the story block in the list of blocks.

@enejb can you confirm you were testing on iOS? If so, this block is still not out for production.

👋 I was testing the 1.48.1 betafix on iOS and the Story block is available in the inserter menu, I'm not sure if that changed recently 🤔 .

In fact I tested a bit that block and I found some issues (you can check them in this comment), one of them produce a crash so probably we should review them and potentially make another betafix. What do you think?

@enejb @geriux

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.