Skip to content

Integrated updates to both WordPressKit and WordPressAuthenticator - #10916

Merged
2 commits merged into
developfrom
pods/update-wpkit-wpauth
Jan 31, 2019
Merged

Integrated updates to both WordPressKit and WordPressAuthenticator#10916
2 commits merged into
developfrom
pods/update-wpkit-wpauth

Conversation

@ghost

@ghost ghost commented Jan 30, 2019

Copy link
Copy Markdown

Description

Propagates the fix for this issue in WordPressKit to WPiOS. Because this was an update to the major version, a corresponding update to WordPressAuthenticator was also required.

Testing

Checkout the branch and confirm that existing tests pass.

To test:

  • Checkout the branch and confirm that existing tests pass.
  • Perform a smoke test of the app and confirm nothing looks off. You should be able to exercise network requests without issue.

Update release notes:

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

@ghost ghost added the Network/Sync label Jan 30, 2019
@ghost ghost added this to the 11.8 milestone Jan 30, 2019
@ghost ghost self-assigned this Jan 30, 2019
@ghost
ghost requested a review from yaelirub January 30, 2019 00:46
@ghost

ghost commented Jan 30, 2019

Copy link
Copy Markdown
Author

At the moment, Circle CI is failing with this error:

screen shot 2019-01-30 at 7 07 51 am

@ghost

ghost commented Jan 30, 2019

Copy link
Copy Markdown
Author

@frosty if you have a moment, would be interested in checking out this PR, which propagates the locale-handling change we introduced to WordPressKit?

@frosty

frosty commented Jan 31, 2019

Copy link
Copy Markdown
Contributor

@stevebaranski I restarted Circle and it's passing now 👍

@frosty frosty left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

✅ Builds and tests pass
✅ Smoke tested the app in multiple languages – looks good!

@ghost
ghost merged commit ba58df4 into develop Jan 31, 2019
@ghost
ghost deleted the pods/update-wpkit-wpauth branch January 31, 2019 13:34
This pull request was closed.
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.

2 participants