Skip to content

Integrates new versions of WPKit and WPAuth to fix a signup issue.#13585

Merged
jkmassel merged 5 commits intorelease/14.2.1from
try/fix-email-checking
Mar 3, 2020
Merged

Integrates new versions of WPKit and WPAuth to fix a signup issue.#13585
jkmassel merged 5 commits intorelease/14.2.1from
try/fix-email-checking

Conversation

@diegoreymendez
Copy link
Copy Markdown
Contributor

@diegoreymendez diegoreymendez commented Mar 3, 2020

Description

Fixes #13583

This PR improves the error handling so that we don't ignore some error responses.

Testing:

Test 1:

Try signing up with a valid email account. It should send you the magic link.

Test 2:

Try signing up with an invalid email provider. For example "tester@mailinator.com".
You should see an error prompting you to use a different email provider.

Test 3:

Try singing up with a used email address (you can try with the email of your existing account).
You should see an error letting you know the email address is not available.

PR submission checklist:

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

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Mar 3, 2020

You can trigger an installable build for these changes by visiting CircleCI here.

@peril-wordpress-mobile
Copy link
Copy Markdown

peril-wordpress-mobile bot commented Mar 3, 2020

Warnings
⚠️ PR is not assigned to a milestone.

Generated by 🚫 dangerJS

@jkmassel jkmassel marked this pull request as ready for review March 3, 2020 16:56
@jkmassel jkmassel self-requested a review March 3, 2020 16:58
@jkmassel jkmassel self-requested a review March 3, 2020 19:13
Copy link
Copy Markdown
Contributor

@jkmassel jkmassel left a comment

Choose a reason for hiding this comment

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

I tested according to the test plan, and everything works as expected.

Also – we're referencing all pods by version, so the code is good to go.

Will merge when green.

@jkmassel jkmassel merged commit b64fd2e into release/14.2.1 Mar 3, 2020
@jkmassel jkmassel deleted the try/fix-email-checking branch March 3, 2020 20:16
@diegoreymendez
Copy link
Copy Markdown
Contributor Author

Thanks @jkmassel !

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