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.

Sign up for GitHub

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

Jump to bottom

Release/1.2.1 #876

Merged
merged 2 commits into from
Apr 17, 2019
Merged

Release/1.2.1 #876

merged 2 commits into from
Apr 17, 2019

Conversation

jtreanor
Copy link
Contributor

This PR merges release/1.2.1 into master.

Related PR on WPAndroid: wordpress-mobile/WordPress-Android#9638
Related PR on Aztec Android: wordpress-mobile/AztecEditor-Android#810

iOS should not be affected, although new bundles were created should remain the same.

@jtreanor jtreanor requested a review from mzorz April 16, 2019 15:04
@peril-wordpress-mobile
Copy link

Warnings
⚠️ PR is missing at least one label.
⚠️ PR is not assigned to a milestone.
⚠️ PR has more than 500 lines of code changing. Consider splitting into smaller PRs if possible.

Generated by 🚫 dangerJS

mzorz
mzorz approved these changes Apr 17, 2019
View reviewed changes
Copy link
Contributor

@mzorz mzorz left a comment

Choose a reason for hiding this comment

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

LGTM! :shipit:

@mzorz mzorz mentioned this pull request Apr 17, 2019
Closed
19 tasks
@jtreanor jtreanor merged commit 1584775 into master Apr 17, 2019
@jtreanor jtreanor deleted the release/1.2.1 branch April 17, 2019 13:19
This was referenced Apr 17, 2019
Merged
Merged
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mzorz mzorz mzorz approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@jtreanor @mzorz