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

Update Aztec to version 1.7.0 and GB latest (1.7.0) #11885

Merged
merged 11 commits into from
Jun 14, 2019

Conversation

SergioEstevao
Copy link
Contributor

To test:

  • Test copy paste from other apps and internally in GB
  • Test insertion and use of all blocks
  • Check focus issues
  • Check editing the Title of the post and press Enter

Note: still need to update the release notes, waiting for the final version GB-mobile to know what changed.

Update release notes:

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

@SergioEstevao SergioEstevao added the Gutenberg Editing and display of Gutenberg blocks. label Jun 10, 2019
@SergioEstevao SergioEstevao added this to the 12.7 milestone Jun 10, 2019
Copy link
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.

I spotted a couple of issues, but none of those seem to be new, or generated by recent changes on Aztec.
I'm happy to ✅ if these issues will be handled for the future release.

These are the issues in case they are not being tracked:

- UI Api called from background thread when canceling an image upload:

 Main Thread Checker: UI API called on a background thread: -[UIApplication statusBarFrame]
PID: 6653, TID: 2439575, Thread name: (none), Queue name: NSManagedObjectContext 0x28246fc00, QoS: 0
Backtrace:
4   WordPress                           0x0000000103986f40 -[TracksDeviceInformation statusBarHeight] + 72
5   WordPress                           0x000000010398f988 -[TracksService mutableDeviceProperties] + 1016
6   WordPress                           0x000000010398d304 -[TracksService trackEventName:withCustomProperties:] + 488
7   WordPress                           0x0000000102bafcd4 -[WPAnalyticsTrackerAutomatticTracks track:withProperties:] + 1136
8   WordPress                           0x0000000102baf858 -[WPAnalyticsTrackerAutomatticTracks track:] + 56
9   WordPressShared                     0x0000000107da6d28 +[WPAnalytics track:] + 308
10  WordPress                           0x0000000102c17a58 +[WPAppAnalytics track:] + 60
11  WordPress                           0x0000000102bf2830 -[MediaService trackUploadError:] + 116

This is NOT an app crash, but it might randomly actually crash it.

- Merge text into an indented list block results in the cursor jumping to the indented list item:

This is also happening on Web, it might be a desired behavior.

- Block validation Error on Quote block when starting typing on the second line.

  • The quote block starts with two empty lines
  • Write something in the second leaving the first one empty
  • Change to HTML mode
  • Block validation will throw an error

quote-broken

@SergioEstevao
Copy link
Contributor Author

Block validation Error on Quote block when starting typing on the second line.

This one looks bad, need to investigate more if there is a workaround for it.

@pinarol
Copy link
Contributor

pinarol commented Jun 11, 2019

This one looks bad, need to investigate more if there is a workaround for it.

was this solved @SergioEstevao ? I still see the validation error

@pinarol
Copy link
Contributor

pinarol commented Jun 11, 2019

was this solved @SergioEstevao ? I still see the validation error

Sorry for the wrong alarm, I don't see it anymore

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! 🎉

I found a few minor issues but don't think they are blockers

  1. The caret position remains at start after I paste. Happens only on empty blocks.

copy-paste

  1. Change detection works incorrectly, I see the "You have unsaved changes" action sheet even if I don't change anything:

unsaved-change

@mkevins
Copy link
Contributor

mkevins commented Jun 11, 2019

The caret position remains at start after I paste. Happens only on empty blocks.

I'm observing the same on WordPress-Android.

@etoledom
Copy link
Contributor

Change detection works incorrectly, I see the "You have unsaved changes" action sheet even if I don't change anything:

@pinarol does your test post have an image with caption?

I only was able to reproduce it when I added an image with multi paragraph caption. Removing it the problem is gone.

There might be other instances that generate this issue too.

@pinarol
Copy link
Contributor

pinarol commented Jun 13, 2019

@etoledom 👋 here's my post:

<!-- wp:paragraph -->
<p>Sdfaas</p>
<!-- /wp:paragraph -->

<!-- wp:video {"id":4482} -->
<figure class="wp-block-video"><video controls src="https://eritreocazzulati.files.wordpress.com/2019/06/20190611_151006.mp4"></video></figure>
<!-- /wp:video -->

<!-- wp:paragraph -->
<p>Asdf</p>
<!-- /wp:paragraph -->

<!-- wp:quote -->
<blockquote class="wp-block-quote"><p>asdasd asd asdf</p><cite>asdfsd sade asd<br />asdfdfs<br />asdf<br />asdfasd<br />High<br />Asdf<br />Asdf<br />Asdf</cite></blockquote>
<!-- /wp:quote -->

<!-- wp:quote -->
<blockquote class="wp-block-quote"><p></p></blockquote>
<!-- /wp:quote -->

<!-- wp:video {"id":1598} -->
<figure class="wp-block-video"><video controls src="https://videos.files.wordpress.com/4TucFubg/vid_20190603_122736-1_hd.mp4"></video></figure>
<!-- /wp:video -->

<!-- wp:video {"id":1490} -->
<figure class="wp-block-video"><video controls src="https://videos.files.wordpress.com/2WOwVmnA/ytzfjbuqce-1_hd.mp4"></video></figure>
<!-- /wp:video -->

<!-- wp:video {"id":1637} -->
<figure class="wp-block-video"><video controls src="https://videos.files.wordpress.com/XWdEP3Np/20190611_151006-1_hd.mp4"></video></figure>
<!-- /wp:video -->

<!-- wp:video -->
<figure class="wp-block-video"></figure>
<!-- /wp:video -->

<!-- wp:image {"id":1623} -->
<figure class="wp-block-image"><img src="https://pinpin.blog/wp-content/uploads/2019/06/Screen-Shot-2019-04-03-at-18.40.11.png" alt="" class="wp-image-1623"/></figure>
<!-- /wp:image -->

<!-- wp:image {"id":1592} -->
<figure class="wp-block-image"><img src="https://pinpin.blog/wp-content/uploads/2019/06/y2mate-1.com-nature_short_video_iqx4s3apkqm_240p_dvd.original.jpg" alt="" class="wp-image-1592"/></figure>
<!-- /wp:image -->

Very interesting thing happened: I removed the last block with {"id":1592} and updated the post, problem stopped occurring. When I add it back it started happening again. So I thought there's a problem related with that specific block, but after a few more publishes now I have the block {"id":1592} but I can't see the problem :( Maybe it is not directly related with the post content. Note: I sometimes edit this post from Android also.

@etoledom
Copy link
Contributor

@pinarol thanks for sharing!

Maybe it is not directly related with the post content.

Yeah that's also a possibility, or at least not related to a specific kind of content.

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.

I can't seem to find any other problem, LGTM! 👍

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

Looks good!

@pinarol pinarol merged commit 8c1c403 into develop Jun 14, 2019
@pinarol pinarol deleted the issue/update_gb_to_latest_version branch June 14, 2019 12:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Gutenberg Editing and display of Gutenberg blocks.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants