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.33.0 #2493

Merged
merged 74 commits into from
Jul 23, 2020
Merged

Release 1.33.0 #2493

merged 74 commits into from
Jul 23, 2020

Conversation

SergioEstevao
Copy link
Contributor

@SergioEstevao SergioEstevao commented Jul 20, 2020

Release for Gutenberg Mobile 1.33.0

Related PRs

Extra PRs that Landed After the Release Was Cut

Changes

  • [***] Media editing support in Media & Text block.
  • [***] New block: Social Icons
  • [*] Cover block placeholder is updated to allow users to start the block with a background color

Test plan

  • Use the main WP apps to test the changes above.
  • Check WPAndroid and WPiOS PRs if there are specific tests to run.
  • Smoke test the main WP apps for general writing flow.

Release Submission Checklist

  • Release number was bumped in package.json and gutenberg/packages/react-native-editor/package.json.
  • Aztec dependencies are pointing to a stable release.
    • iOS: 'grep WordPressAztec-iOS RNTAztecView.podspec'.
    • iOS: Aztec dependencies match in RNTAztecView.podspec and gutenberg/packages/react-native-aztec/RNTAztecView.podspec.
    • Android: 'grep aztecVersion react-native-aztec/android/build.gradle'.
  • Gutenberg 'Podfile' and 'Podfile.lock' inside 'gutenberg/package/react-native-editor/ios/' are updated to the release number.
  • Bundle package of the release is updated.
  • Check if RELEASE-NOTES.txt and gutenberg/packages/react-native-editor/RELEASE-NOTES.txt are updated with all the changes that made it to the release.

SergioEstevao and others added 30 commits June 17, 2020 09:30
Change the bundle step to only create updated bundles and not get
updates strings from the web.
This way for code updates only we just need to run yarn bundle.
For the releases we run yarn release:bundle.
This will accelerate the builds for WP-Android and for WP-iOS it will
allow deploy and test of PR that need bundle updates quicker.
…main

Update Release doc references to main
Running the pod install could changes the lock file inside the GB repo
so we need to run that before we commit the change in Gutenberg.
@peril-wordpress-mobile
Copy link

peril-wordpress-mobile bot commented Jul 20, 2020

Wanna run full suite of Android and iOS UI tests? Click here and 'Approve' CI job!

@SergioEstevao SergioEstevao requested a review from mchowning July 20, 2020 10:56
@SergioEstevao SergioEstevao added this to the 1.33 milestone Jul 20, 2020
@mkevins
Copy link
Contributor

mkevins commented Jul 21, 2020

I've done the following tests on Android (Pixel 3a) via this apk: https://62039-9306568-gh.circle-artifacts.com/0/Artifacts/WordPress-pr-12480-build-62039.apk

Everything is working smoothly 🎉 The only minor note is that in one test case (group.md#tc008), the behavior differs from test, but I think the test case needs to be updated, since the behavior seems correct to me.

Sanity tests - Android:

DarkMode-1

  • Gallery block - Dark mode - steps
  • Shortcode block - Dark mode - steps
  • Media Text block - Dark mode - steps

Group-1

  • Group - AppenderButton is rendered - steps
  • Group - Check if Group placeholder is visible for the unselected state - steps
  • Group - Check if Group placeholder is render in nested structure - steps
  • Group - Nested block have proper border styling - steps
  • Group - Nested block have proper margins values - steps
  • ❓ Group - Breadcrumbs on FloatingToolbar is properly displayed - steps

The behavior differs slightly from the test case expectation here, but I think the app's behavior is correct, and perhaps the test case merely needs to be updated (maybe the images too, or just a note that this has changed?).

Expect only selected block title is visible in Breadcrumbs on root list level

since block icon is shown at root level (and the navigate up icon is not shown).

Spacer-1

  • Spacer block - Spacer is rendered without crash - steps
  • Spacer block - Settings: Control spacer height using the slider - steps
  • Spacer block - Settings: Control spacer height using the text input - steps

Buttons-1

  • Buttons block - Buttons block is added with focused Button with gray background color and white text color - steps
  • Buttons block - Buttons inline appender - steps
  • Buttons block - Add another Button inside Buttons using inserter - steps
  • Buttons block - Button's wrapper grows properly - steps

Buttons-2

  • Buttons block - Removing Button along with Buttons block - steps
  • Buttons block - Removing exactly one Button (when Buttons contain more of them) - steps

Buttons-3

  • Buttons block - Wrapping Buttons - steps
  • Buttons block - Buttons alignment options - steps

Button-6

  • Buttons block - Settings: Open in new tab - steps
  • Buttons block - Settings: Link rel - steps
  • Buttons block - Settings: Link URL - steps
  • Buttons block - Settings: Remove link - steps
  • Buttons block - Settings: Synchronize with button options - steps

@lukewalczak
Copy link
Contributor

lukewalczak commented Jul 21, 2020

Sanity tests - iOS

Gallery-1

  • Gallery block - Close/Re-open post with an ongoing image upload - steps
  • Gallery block - Close post with an ongoing image upload - steps

Gallery-2

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

Gallery-3

  • Gallery block - Add Gallery Caption - steps

NOTE: When have selected one of text style it works as long as the formatted word is not removed. After removing styling control is highlighted but the text is not styled.

ezgif com-resize (3)

  • Gallery block - Add Gallery Image Caption - steps

🔴 break line doesn't work.

  • Gallery block - Try adding same images from WP Media library and moving the images around - steps

❓ I have one question: after test case Insert image from device (failing) I left in my draft photos which uploading failed and when I generate a preview I've noticed these photos are "displayed" (as a
white block with border) in the post. Is it intentional?

draft preview
IMG_3899 IMG_3900

Columns - 2

  • Columns - Max limit of Column number - steps
  • Columns - Min limit of Column number - steps
  • Columns - Removing Column with remove button - steps
  • Columns - Removing the only one left Column with remove button - steps
  • Columns - Changing alignment on Columns - steps
  • Columns - Changing alignment on individual Columnn - steps
  • Columns - Alignment attribute is properly passed when adding Column - steps
  • Columns - Deep nesting is possible (iOS only)- steps

Columns - 3

  • Columns - Check if Column placeholders is visible for the unselected state - steps
  • Columns - Appearance in vertical device position (vertical layout) - steps
  • Columns - Appearance in horizontal device position (horizontal layout) - steps
  • Columns - Mover direction - steps
  • Columns - Check if Column placeholder is render in nested structure - steps
  • Columns - Unselected Column placeholder is streched to content in nested structure- steps

Columns - 4

  • Columns - Check if in DarkMode all components gets proper colors - steps

MediaText-3

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

MediaText-4

  • Media Text block - Media & Text alignment - steps
  • Media Text block - Vertical alignment - steps

MultipleUpload-1

  • Simultaneous uploads - steps

Cover-1

  • Cover - Add image background from WordPress Media library - steps
  • Cover - Change Minimum height in pixels - steps
  • Cover - Change Background Opacity - steps
  • Cover - Image focal point is rendered properly - steps
  • Cover - Custom color is rendered properly - steps

Editor Theme - 1

  • Default Colors - Check that default colors still load - steps
  • Default Gradients - Check that default gradients still load - steps
  • Custom Colors - Check that custom colors load in the editor - steps

Social Icons - 1

  • The newly created Social Icons block is added with 4 icons - steps
  • Active icon gets product color - steps
  • Only active icons are visible when not selected - steps

Social Icons - 2

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

@pinarol
Copy link
Contributor

pinarol commented Jul 21, 2020

NOTE: When have selected one of text style it works as long as the formatted word is not removed. After removing styling control is highlighted but the text is not styled.

That's a known issue(added this to the test steps):

#2154
#2337

Gallery block - Add Gallery Image Caption - steps
🔴 break line doesn't work.

I lost track of how the line break was supposed to work in captions, I am happy to update the test case accordingly if somebody tells me the latest state. @SergioEstevao knows maybe?

❓ I have one question: after test case Insert image from device (failing) I left in my draft photos which uploading failed and when I generate a preview I've noticed these photos are "displayed" (as a
white block with border) in the post. Is it intentional?

That's a good question, I think it is one of those things we don't have a complete win-win scenario. If we strip out failed media then we take away the ability to retry the upload easily. If we leave it this way then we'll see empty tiles because the media URL is the local url. IMO since the user clearly sees that the upload is failed they can easily make sense about why they see empty tiles. But I agree that this is not super ideal. I just don't have a better suggestion.

@SergioEstevao
Copy link
Contributor Author

I have one question: after test case Insert image from device (failing) I left in my draft photos which uploading failed and when I generate a preview I've noticed these photos are "displayed" (as a
white block with border) in the post. Is it intentional?

Yep, because the preview is done on the server, and those images that failed are represented with local urls, this will happen.

I lost track of how the line break was supposed to work in captions, I am happy to update the test case accordingly if somebody tells me the latest state. @SergioEstevao knows maybe?

So the current method (replicate the web) is that break lines don't work in the middle of the caption paragraphs, and if you do a break at the end of the caption text, it will insert a new paragraph bellow the block.

@mchowning
Copy link
Contributor

mchowning commented Jul 21, 2020

Writing Flow tests on Android

TLDR: Only issues I found were edge cases that existed before this release (noted below), so I don't see anything that would block the release. 🎉

  • TC001 - Block insertion flow
  • TC001 - Paste formatted text copied from website
  • TC001 - Multiline components
    Noted an issue on Android where tapping enter within a quote or pullquote citation both split the block and added a new line to the citation, but it is not a regression: Tapping Enter in citations to split block also adds a new line #2498
  • 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

@mchowning
Copy link
Contributor

Writing Flow tests on iOS

Found no new issues 🎉

  • 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

@mchowning
Copy link
Contributor

Looks like we've run all our sanity tests and there are no issues to block the release. 🙂

cc: @SergioEstevao

Copy link
Contributor

@pinarol pinarol left a comment

Choose a reason for hiding this comment

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

LGTM! 🎉

@SergioEstevao SergioEstevao merged commit 5d3172c into main Jul 23, 2020
@SergioEstevao SergioEstevao deleted the release/1.33.0 branch July 23, 2020 10:20
@SergioEstevao
Copy link
Contributor Author

For the record, I tested the folowing in WP-iOS and they are working correctly:

  • Media editing support in Media & Text block.
  • Cover block placeholder is updated to allow users to start the block with a background color

@mchowning
Copy link
Contributor

I tested the following on WPAndroid:

  • Media editing support in Media & Text block.
  • Cover block placeholder is updated to allow users to start the block with a background color
  • Social Icons

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.

8 participants