Skip to content

Conversation

@hypest
Copy link
Contributor

@hypest hypest commented Apr 13, 2019

This PR brings in a newer version of Aztec, to include the fix for the list block writing flow. See the gutenberg-mobile PR for details.

To test:

  • In a post with blocks, use a list block and follow the steps in the gutenberg-mobile PR

Update release notes:

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

Copy link
Contributor

@daniloercoli daniloercoli left a comment

Choose a reason for hiding this comment

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

LGTM! Maybe restart the workflows that are failing? Don't have the permissions to do that.

@hypest
Copy link
Contributor Author

hypest commented Apr 15, 2019

Maybe restart the workflows that are failing?

Tried that a few times already to no avail. Still investigating...

@hypest hypest requested a review from marecar3 April 15, 2019 16:32
@hypest
Copy link
Contributor Author

hypest commented Apr 15, 2019

👋 @marecar3 , I've added you as a reviewer since you might have a chance to do it shortly (timezones FTW :) ). Danilo has already reviewed and accepted the changes for the list block related part. I've added a fix for the JitPack issue (not being able to produce the Android JS bundle) that was blocking the CircleCI build. The gutenberg-mobile side fix is in the wordpress-mobile/gutenberg-mobile#873 PR.

@marecar3
Copy link
Contributor

Hey @hypest I will take a look at it :)

Copy link
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!

@hypest hypest merged commit b06bd69 into develop Apr 17, 2019
@hypest hypest deleted the gb/list-caret-at-end-fix branch April 17, 2019 07:23
@hypest hypest mentioned this pull request Apr 17, 2019
1 task
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.

4 participants