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

Invalid Authentication Link Alert #1606

Merged
merged 7 commits into from
Jul 8, 2024

Conversation

jleandroperez
Copy link
Contributor

@jleandroperez jleandroperez commented Jun 27, 2024

Fix

In this PR we're building a UI, to be presented whenever the user attempts to authenticate using a link that's no longer valid.

Closes #1605

Test: From Onboarding

  1. Fresh install Simplenote iOS
  2. Make sure you're logged out!
  3. Switch to Safari and open: simplenote://login?auth_key=1234&auth_code=5678
  • Verify Simplenote ends up in the Foreground
  • Verify that shortly after you get an alert, indicating that the Magic Link isn't valid
  • Verify that pressing the Request new Link button gets you back into the Login UI

Test: From Login UI!

  1. Fresh install Simplenote iOS
  2. Make sure you're logged out!
  3. Press on the Log In button
  4. Switch to Safari and open: simplenote://login?auth_key=1234&auth_code=5678
  • Verify that the Invalid Link Alert is also presented
  • Verify that pressing the Request new Link button gets you back into the Login UI

Test: From Login UI!

  1. Fresh install Simplenote iOS
  2. Make sure you're logged out!
  3. Press on the Sign Up button
  4. Switch to Safari and open: simplenote://login?auth_key=1234&auth_code=5678
  • Verify that the Invalid Link Alert is also presented
  • Verify that pressing the Request new Link button gets you back into the Login UI

Release

These changes do not require release notes.

@jleandroperez jleandroperez self-assigned this Jun 27, 2024
@jleandroperez jleandroperez marked this pull request as ready for review June 27, 2024 18:55
@wpmobilebot
Copy link
Collaborator

wpmobilebot commented Jun 27, 2024

You can test the changes in simplenote-ios from this Pull Request by:

  • Clicking here or scanning the QR code below to access App Center
  • Then installing the build number pr1606-eb62589-0190745a-1e5b-4029-9085-d667ab85756c on your iPhone

If you need access to App Center, please ask a maintainer to add you.


// MARK: - MagicLinkConfirmationView
//
struct MagicLinkInvalidView: View {
Copy link
Contributor

Choose a reason for hiding this comment

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

I think there is some duplication here with the invalid view and the check your email view. They are the same except that the invalid view has the new link button. Not sure if it is worth it, but I wonder if there is a way to save on the duplication here

Copy link
Contributor

@charliescheer charliescheer left a comment

Choose a reason for hiding this comment

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

👍

@jleandroperez jleandroperez added the enhancement Improve existing functionality. label Jul 2, 2024
Base automatically changed from add/magic-link-auth to trunk July 8, 2024 19:40
@dangermattic
Copy link
Collaborator

1 Warning
⚠️ PR is not assigned to a milestone.

Generated by 🚫 Danger

@jleandroperez
Copy link
Contributor Author

RE: MagicLinkView(s), we'll need to revisit the UX for this flow. Following up in another PR.

Thank you Charlie!!

@jleandroperez jleandroperez merged commit 2614941 into trunk Jul 8, 2024
12 of 13 checks passed
@jleandroperez jleandroperez deleted the lantean/1605-invalid-link-alert branch July 8, 2024 19:43
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement Improve existing functionality.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Magic Login Authentication
4 participants