Skip to content

Release/1.3.0#895

Merged
hypest merged 146 commits intomasterfrom
release/1.3.0
Apr 22, 2019
Merged

Release/1.3.0#895
hypest merged 146 commits intomasterfrom
release/1.3.0

Conversation

@jtreanor
Copy link
Copy Markdown
Contributor

@jtreanor jtreanor commented Apr 19, 2019

This merges release/1.3.0 to master for the next releases of the WPiOS/WPAndroid apps.

Related PR on WPiOS: wordpress-mobile/WordPress-iOS#11502
Related PR on Aztec iOS: wordpress-mobile/AztecEditor-iOS#1176
Related PR on WPAndroid: wordpress-mobile/WordPress-Android#9668
Related PR on Aztec Android: N/A

Tug and others added 30 commits April 2, 2019 14:37
@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Apr 19, 2019

Warnings
⚠️ PR is missing at least one label.
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS

Copy link
Copy Markdown
Contributor

@etoledom etoledom left a comment

Choose a reason for hiding this comment

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

My tests on iOS side look good! ✅

Copy link
Copy Markdown
Contributor

@marecar3 marecar3 left a comment

Choose a reason for hiding this comment

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

LGTM!

Even we found a couple of issues on Android when using List Block, it's ok to merge this one as iOS doesn't have those kinds of issues.

@marecar3
Copy link
Copy Markdown
Contributor

marecar3 commented Apr 19, 2019

Removing approve as maybe we should consider disabling List Block on Android until we fix some of the issues that we found on the release day :

Here is the list of bugs regarding List Block :

  1. Half of word is gone after text is broken into two parts
  2. Hitting return twice doesn't escape block + create next one
  3. Undo removes whole List Block
  4. Single digit numbers are moving while List is editing
  5. List block - performance issues

cc: @jtreanor , @etoledom , @hypest

@marecar3 marecar3 self-requested a review April 19, 2019 22:56
@hypest
Copy link
Copy Markdown
Contributor

hypest commented Apr 20, 2019

Yes, as we've been chatting in Slack, let's ship the List block for iOS but hold it back for Android until the important issues are fixed.

@hypest hypest merged commit 136ab34 into master Apr 22, 2019
@hypest hypest deleted the release/1.3.0 branch April 22, 2019 09:49
@mkevins mkevins restored the release/1.3.0 branch May 14, 2019 04:09
@mkevins mkevins deleted the release/1.3.0 branch May 14, 2019 04:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.