Skip to content
This repository was archived by the owner on Sep 15, 2025. It is now read-only.

Conversation

@ScoutHarris
Copy link
Contributor

@ScoutHarris ScoutHarris commented Aug 20, 2019

Description

This adds a method to create a WordPress account using Apple ID credentials.

Fixes #n/a

Testing Details

Can be tested with WPiOS PR: wordpress-mobile/WordPress-iOS#12350
Also related to WPAuth PR: wordpress-mobile/WordPressAuthenticator-iOS#116

  • Please check here if your pull request includes additional test coverage.

Copy link
Contributor

@frosty frosty left a comment

Choose a reason for hiding this comment

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

One small comment on this one, otherwise all looks good 👍

*
* @param token Token provided by Apple.
* @param email Apple email to use for new account.
* @param userName The username for the new account. Formed from the Apple ID fullname.
Copy link
Contributor

@frosty frosty Aug 21, 2019

Choose a reason for hiding this comment

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

I think this is the user's name, not their username? If so, could we update the comment to make this clear?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Gah. Clearly I had the endpoint param name on my brain. I'll change it. Thanks.

@ScoutHarris ScoutHarris merged commit 96ba9cc into develop Aug 21, 2019
@ScoutHarris ScoutHarris deleted the feature/create_account_with_apple branch August 21, 2019 18:31
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants