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 v1.17 #1566

Merged
merged 239 commits into from
Nov 15, 2019
Merged

Release v1.17 #1566

merged 239 commits into from
Nov 15, 2019

Conversation

hypest
Copy link
Contributor

@hypest hypest commented Nov 11, 2019

Release for Gutenberg Mobile v1.17.0

Related PRs:

Changes:

Test plan

  1. Please use the main WP apps to test the changes above. See WPAndroid and WPiOS PRs for a tests checklist.
  2. Smoke test the main WP apps for general writing flow

Update release notes:

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

marecar3 and others added 30 commits September 30, 2019 13:14
Increment gb-mobile version to 1.14.0
Update bundle files
# Conflicts:
#	package.json
…pens_safari

Don't allow interactions with URL in Aztec text blocks.
# Conflicts:
#	gutenberg
…Content-is-missing-on-long-posts-in-html-mode

Fix/1268 content is missing on long posts in html mode
# Conflicts:
#	bundle/android/App.js
#	bundle/android/App.js.map
#	bundle/ios/App.js
#	bundle/ios/App.js.map
#	gutenberg
#	react-native-aztec/ios/Cartfile
#	react-native-aztec/ios/Cartfile.resolved
#	react-native-gutenberg-bridge/android/src/main/java/org/wordpress/mobile/WPAndroidGlue/WPAndroidGlueCode.java
@marecar3
Copy link
Contributor

Let's make sure that this PR: #1582
gets merged.

@hypest
Copy link
Contributor Author

hypest commented Nov 15, 2019

I've tested the WPAndroid and WPiOS PRs and this looks good to me 👍!

@marecar3
Copy link
Contributor

marecar3 commented Nov 15, 2019

Hey, @hypest @mchowning and @cameronvoell WPAndroid look nice, but I have found some regressions.

  1. Image upload (visual) regression - when a placeholder is presented on the end of upload (when the progress is 100%), the progress bar is on the bottom of the placeholder and it should be on top of the placeholder.

ezgif com-video-to-gif (11)

  1. Video upload (visual) regression - when the upload is in progress placeholder isn't good as a background is missing and also image in the center should be bigger (similar to the image placeholder)

ezgif com-video-to-gif (12)

It seems that they aren't blockers for a merge 🤔

@mchowning
Copy link
Contributor

mchowning commented Nov 15, 2019

Nice catches @marecar3 !

Image upload (visual) regression - when a placeholder is presented on the end of upload (when the progress is 100%), the progress bar is on the bottom of the placeholder and it should be on top of the placeholder.

I see this as well on both platforms. It does look like a regression.

Video upload (visual) regression - when the upload is in progress placeholder isn't good as a background is missing and also image in the center should be bigger (similar to the image placeholder)

I'm not as certain about this one as @iamthomasbishop recently opened an issue saying that the image placeholder icon is too big and that it looks like the video block has the proper styling. We should probably get his feedback before doing any changes on this (for the next release).

It seems that they aren't blockers for a merge 🤔

I think that's right. I don't see any reason either of these should block the release.

@iamthomasbishop
Copy link
Contributor

I'm not as certain about this one as @iamthomasbishop recently opened an issue saying that the image placeholder icon is too big and that it looks like the video block has the proper styling. We should probably get his feedback before doing any changes on this (for the next release).

Yes, you're right, the image block's placeholder icon is incorrect – video uses the proper style.

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 via both WPAndroid and WPiOS and everything looks good.

@mchowning
Copy link
Contributor

Updated from 9c19f9c to bd25fb8 because I was getting a different bundle when building it locally. Looks like that was just because bundles include references to the local environment, so reverted back to 9c19f9c.

@mchowning mchowning merged commit c6d095e into master Nov 15, 2019
@marecar3 marecar3 deleted the release/1.17 branch March 20, 2020 09:56
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.