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

1.24.0 Release -> Master #2025

Merged
merged 240 commits into from
Mar 20, 2020
Merged

1.24.0 Release -> Master #2025

merged 240 commits into from
Mar 20, 2020

Conversation

marecar3
Copy link
Contributor

@marecar3 marecar3 commented Mar 17, 2020

Release for Gutenberg Mobile v1.24.0

Related PRs

Extra PRs that Landed After the Release Was Cut

Changes

#1931 New block: Latest Posts
#1953 Fix Quote block's left border not being visible in Dark Mode
#1987 Added Starter Page Templates: when you create a new page, we now show you a few templates to get started more quickly.
#1964 Fix crash when pasting HTML content with embeded images on paragraphs

PR Submission Checklist

  • I have considered adding unit tests where possible.
  • I have considered if this change warrants user-facing release notes and have added them to RELEASE-NOTES.txt if necessary.

Test plan

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

geriux and others added 30 commits January 17, 2020 15:42
This reverts commit 9d45f09.
…e into add/cover-block

# Conflicts:
#	gutenberg
…e into add/cover-block

# Conflicts:
#	gutenberg
…e into add/cover-block

# Conflicts:
#	gutenberg
iOS 13.0 on SauceLabs and 13.3 locally.
* improve test:e2e:ios:local, skip bundling on CI since we do it manualy

* use already existing command

* run preios before tests build and use --repo-update flag
…e-aztec-old-submodule

Remove react-native-aztec-old-submodule
@guarani
Copy link
Contributor

guarani commented Mar 19, 2020

@marecar3 while retesting #2025 (comment) I can reproduce consistently an error where image URLs are saved as file:https:/// instead of https:// in Gallery TC008. When this happens, the images stay slightly blurry and tapping on them reveals an iOS action sheet with no options (just a title).

I reproduced it on about 3 of 4 attempts with this particular test (Gallery TC008). I ran similar tests such as TC005 and TC007 twice each and I didn't come across this error.

@chipsnyder
Copy link
Contributor

chipsnyder commented Mar 19, 2020

It looks like WordPress/gutenberg#20832 broke the Latest-Posts Block. Working on a fix for that now.

Edit PR Created #2039

@chipsnyder
Copy link
Contributor

RE: #2025 (comment)
Just merged the PRs for this:
WordPress/gutenberg#21025 -> 8017d0f3526015529f20067b57c67953a9980bdd

#2039 -> cec2df78aa674084e57870e007a1858076ab2ecb

@cameronvoell
Copy link
Contributor

cameronvoell commented Mar 19, 2020

Testing Android using WordPress PR APK build 41182 (latest commit 82291f223fed5641da0689a882310b2ddb4d245c)

@marecar3
Copy link
Contributor Author

marecar3 commented Mar 20, 2020

@marecar3 while retesting #2025 (comment) I can reproduce consistently an error where image URLs are saved as file:https:/// instead of https:// in Gallery TC008. When this happens, the images stay slightly blurry and tapping on them reveals an iOS action sheet with no options (just a title).

I reproduced it on about 3 of 4 attempts with this particular test (Gallery TC008). I ran similar tests such as TC005 and TC007 twice each and I didn't come across this error.

Nice work @guarani!!! Thanks for finding this one!

Adding comment from @geriux :
Well, this is not related to the other URL issues. This bug is still happening but I’m not sure if its a blocker or not. Since its a case when the user closes and opens the editor while uploading images to a gallery

if the editor is opened it works fine, but if you close and open several times with a few images uploading some fail

as Paul reported

What do you think @pinarol on this one, is it a blocker?

@pinarol
Copy link
Contributor

pinarol commented Mar 20, 2020

hi @marecar3 👋 I don’t think that’s a blocker, you need to close/open several times to repro that. Our download completion mechanism is not perfect, I bet this was working like this from the beginning. Could you open a separate issue for this and put in Mobile Gutenpages board? Thank you!

@marecar3
Copy link
Contributor Author

hi @marecar3 👋 I don’t think that’s a blocker, you need to close/open several times to repro that. Our download completion mechanism is not perfect, I bet this was working like this from the beginning. Could you open a separate issue for this and put in Mobile Gutenpages board? Thank you!

Thanks, @pinarol! I have raised an issue: #2041
Let me know if I need to update something on it.

Copy link
Contributor

@mchowning mchowning left a comment

Choose a reason for hiding this comment

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

Tested all the changes, including the ones that landed after the branch was cut, and everything looks good to me. 👍

@cameronvoell
Copy link
Contributor

With Dark mode merged I'm seeing an issue where I can't scroll up on a longer post after opening and closing the keyboard of a paragraph block. Hard to see the screen twitch while I try and scroll in my low frame rate GIF, but that's what's happening. I'm testing now to see if this issue is isolated to the Post-Dark mode version of the release.

Steps:

  1. on a longer post that allows scrolling, add a paragraph block at the bottom
  2. let the keyboard open for input on a paragraph block at the bottom of a long post
  3. then hit back to close the keyboard
  4. try and scroll back up, it sort of jumps and doesn’t let you scroll up until you try multiple times. Worse in landscape mode.

cant-scroll

@marecar3
Copy link
Contributor Author

With Dark mode merged I'm seeing an issue where I can't scroll up on a longer post after opening and closing the keyboard of a paragraph block. Hard to see the screen twitch while I try and scroll in my low frame rate GIF, but that's what's happening. I'm testing now to see if this issue is isolated to the Post-Dark mode version of the release.

Steps:

  1. on a longer post that allows scrolling, add a paragraph block at the bottom
  2. let the keyboard open for input on a paragraph block at the bottom of a long post
  3. then hit back to close the keyboard
  4. try and scroll back up, it sort of jumps and doesn’t let you scroll up until you try multiple times. Worse in landscape mode.

cant-scroll

Nice job @cameronvoell!
I also can reproduce this one on my device: Pixel 3, Android 10.

@guarani
Copy link
Contributor

guarani commented Mar 20, 2020

I'm seeing a reasonably reproducible crash (75%) on iOS when adding both an image and a video to a post.

  1. Add an image block and choose an image from the device (preferably a high-res one, as that seems to help reproduce this crash)
  2. Add a video (I chose to record a ~5 second video)
  3. After recording the video and tapping "Use Video", the app crashes

iPhone 7, iOS 13.3.1 and WordPress build 25203

@hypest
Copy link
Contributor

hypest commented Mar 20, 2020

👋 @guarani , can you also check if the issue is there for you on the original/earliest build for the release and also on the previous release, to try and pinpoint if this is a new issue or not? Thanks!

@cameronvoell cameronvoell merged commit d584f8d into master Mar 20, 2020
@Tug Tug deleted the release/1.24.0 branch March 28, 2020 12:13
@guarani
Copy link
Contributor

guarani commented Jun 16, 2020

I'm seeing a reasonably reproducible crash (75%) on iOS when adding both an image and a video to a post.

This issue was fixed via wordpress-mobile/WordPress-iOS#14299

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.

None yet