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.16.2 #1580

Closed
wants to merge 3 commits into from
Closed

Release 1.16.2 #1580

wants to merge 3 commits into from

Conversation

marecar3
Copy link
Contributor

@marecar3 marecar3 commented Nov 14, 2019

Fixes: wordpress-mobile/WordPress-Android#10491

Prepares release 1.16.2

WP-iOS PR: N/A
WP-Android: wordpress-mobile/WordPress-Android#10800
WP-GB PR: WordPress/gutenberg#18527

To test:
Steps to repro - Case A

  • Create a new post
  • Click on the empty paragraph block
  • Add a Page break block
  • Click on the "bin" on the Page break block
  • Boom

Steps to repro - Case B

  • Create a new post
  • Add a Page break block
  • Click on the "bin" on the Page break block
  • This time the block is removed without a crash
  • Add a Page break block
  • Click on the "bin" on the Page break block
  • Boom

Update release notes:

  • If there are user facing changes, I have added an item to RELEASE-NOTES.txt.

@hypest
Copy link
Contributor

hypest commented Nov 14, 2019

After discussing over Slack, decided to not target this fix to v13.6 of the apps. Instead, we'll put it in 13.7 via the v1.17 editor release. @marecar3, please retarget the fix accordingly, thanks!

@marecar3
Copy link
Contributor Author

After discussing over Slack, decided to not target this fix to v13.6 of the apps. Instead, we'll put it in 13.7 via the v1.17 editor release. @marecar3, please retarget the fix accordingly, thanks!

Closing this one in favor of #1582

@marecar3 marecar3 closed this Nov 15, 2019
@marecar3 marecar3 deleted the release/1.16.2 branch November 15, 2019 12:54
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