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

[HOLD for payment 2024-03-05] [$250] iOS Hybrid - Sign in - The app does not auto suggest the magic code from email to sign in #55744

Closed
1 of 8 tasks
vincdargento opened this issue Jan 24, 2025 · 25 comments
Assignees
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor

Comments

@vincdargento
Copy link

vincdargento commented Jan 24, 2025

If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!


Issue was found while executing QA for PR #54755

Version Number: 9.0.89.5
Reproducible in staging?: Yes
Reproducible in production?: Yes
If this was caught on HybridApp, is this reproducible on New Expensify Standalone?: No, reproducible on hybrid only
If this was caught during regression testing, add the test name, ID and link from TestRail: #54755
Email or phone of affected tester (no customers): N/A
Issue reported by: Applause Internal Team
Device used: iPhone 15 iOS 18.1.1
App Component: Other

Action Performed:

  1. Open staging Hybrid app
  2. Select Continue with Email option on the Welcome screen
  3. Enter the email address that is added to your iOS mail app
  4. Tap Next
  5. Open the mail app and wait for the code to arrive, don't open it
  6. Go back to the Hybrid app and make sure that it auto suggests the magic code

Expected Result:

Hybrid app auto suggests the magic code from the email to sign in with, as in ND Standalone app.

Actual Result:

Hybrid app does not auto suggest the magic code from email to sign in

Workaround:

Unknown

Platforms:

  • Android: Standalone
  • Android: HybridApp
  • Android: mWeb Chrome
  • iOS: Standalone
  • iOS: HybridApp
  • iOS: mWeb Safari
  • MacOS: Chrome / Safari
  • MacOS: Desktop

Screenshots/Videos

bug.mp4
bug2.mp4

View all open jobs on GitHub

Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~021884130584906000105
  • Upwork Job ID: 1884130584906000105
  • Last Price Increase: 2025-01-28
  • Automatic offers:
    • brunovjk | Reviewer | 105889014
    • ZhenjaHorbach | Contributor | 106254084
Issue OwnerCurrent Issue Owner: @stephanieelliott
@vincdargento vincdargento added Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 labels Jan 24, 2025
Copy link

melvin-bot bot commented Jan 24, 2025

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@melvin-bot melvin-bot bot added the Overdue label Jan 27, 2025
@stephanieelliott stephanieelliott added the External Added to denote the issue can be worked on by a contributor label Jan 28, 2025
@melvin-bot melvin-bot bot changed the title iOS Hybrid - Sign in - The app does not auto suggest the magic code from email to sign in [$250] iOS Hybrid - Sign in - The app does not auto suggest the magic code from email to sign in Jan 28, 2025
Copy link

melvin-bot bot commented Jan 28, 2025

Job added to Upwork: https://www.upwork.com/jobs/~021884130584906000105

@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 28, 2025
Copy link

melvin-bot bot commented Jan 28, 2025

Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk (External)

@melvin-bot melvin-bot bot removed the Overdue label Jan 28, 2025
@hannojg
Copy link
Contributor

hannojg commented Jan 28, 2025

This screen:

Image

Is part of the old app's code (https://github.com/Expensify/Mobile-Expensify), thus it's not a bug from this code base.

@hannojg
Copy link
Contributor

hannojg commented Jan 28, 2025

Not sure if issues for the old app's code are tracked in this repository?

@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label Jan 28, 2025
Copy link

melvin-bot bot commented Jan 28, 2025

📣 @brunovjk 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job

@brunovjk
Copy link
Contributor

I'm having a hard time reproducing this issue, I still haven't been able to install "mail app" on my simulator.

@mountiny
Copy link
Contributor

@hannojg have you been able to setup the hybrid app and look into this?

@mountiny mountiny added Weekly KSv2 and removed Daily KSv2 labels Jan 31, 2025
Copy link

melvin-bot bot commented Feb 7, 2025

@hannojg @stephanieelliott @mountiny @brunovjk this issue was created 2 weeks ago. Are we close to approving a proposal? If not, what's blocking us from getting this issue assigned? Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks!

@stephanieelliott
Copy link
Contributor

Hey @brunovjk, any luck looking into this yet?

@melvin-bot melvin-bot bot added the Overdue label Feb 11, 2025
@brunovjk
Copy link
Contributor

Not yet @stephanieelliott, I still haven't been able to reproduce it because of the iOS Mail App, today I'll try some alternative ways and let you know. Thanks.

@melvin-bot melvin-bot bot removed the Overdue label Feb 11, 2025
@hannojg
Copy link
Contributor

hannojg commented Feb 21, 2025

Got the hybrid app working and I am able to reproduce! Will look into fixing this now …

@melvin-bot melvin-bot bot added the Overdue label Feb 21, 2025
@hannojg
Copy link
Contributor

hannojg commented Feb 21, 2025

Opened a PR here that fixes the issue:

@brunovjk
Copy link
Contributor

Should I review it @hannojg? Thanks

@melvin-bot melvin-bot bot removed the Overdue label Feb 21, 2025
@hannojg
Copy link
Contributor

hannojg commented Feb 21, 2025

@brunovjk you're a C+ ? Not sure how it works since it's a direct change / PR towards the Mobile-expensify repo. cc @mountiny

Copy link

melvin-bot bot commented Feb 24, 2025

📣 @ZhenjaHorbach 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!

Offer link
Upwork job
Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑‍💻
Keep in mind: Code of Conduct | Contributing 📖

@github-project-automation github-project-automation bot moved this from MEDIUM to Done in [#whatsnext] #quality Feb 24, 2025
@ZhenjaHorbach
Copy link
Contributor

ZhenjaHorbach commented Feb 24, 2025

@stephanieelliott
Can we make a payment here, please ?
And I accepted an offer

@mountiny mountiny reopened this Feb 24, 2025
@mountiny mountiny added Daily KSv2 Awaiting Payment Auto-added when associated PR is deployed to production and removed Weekly KSv2 labels Feb 24, 2025
@stephanieelliott
Copy link
Contributor

@ZhenjaHorbach you're requesting payment for C+ review of https://github.com/Expensify/Mobile-Expensify/pull/13438 right? Or is there a different one?

If it's https://github.com/Expensify/Mobile-Expensify/pull/13438 the payment be queued for the date the PR hits prod+ 7 days.

@ZhenjaHorbach
Copy link
Contributor

ZhenjaHorbach commented Feb 25, 2025

@ZhenjaHorbach you're requesting payment for C+ review of Expensify/Mobile-Expensify#13438 right? Or is there a different one?

If it's Expensify/Mobile-Expensify#13438 the payment be queued for the date the PR hits prod+ 7 days.

Sorry 😅
It's just that this issue was closed for a short period of time
I thought we weren't going to wait for regression period

But now yes
Let's wait !

@stephanieelliott
Copy link
Contributor

No worries! All good 😅, will pay this one out at the end of the regression period

@melvin-bot melvin-bot bot added Overdue Daily KSv2 and removed Daily KSv2 Overdue labels Feb 28, 2025
@stephanieelliott
Copy link
Contributor

I think this should've went out with the 2/26 deploy, which would make payment due tomorrow. Queuing for payment on 3/5

@stephanieelliott stephanieelliott changed the title [$250] iOS Hybrid - Sign in - The app does not auto suggest the magic code from email to sign in [HOLD for payment 2024-03-05] [$250] iOS Hybrid - Sign in - The app does not auto suggest the magic code from email to sign in Mar 5, 2025
@stephanieelliott
Copy link
Contributor

Summarizing payment on this issue:

Upwork job is here: https://www.upwork.com/jobs/~021884130584906000105

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Awaiting Payment Auto-added when associated PR is deployed to production Bug Something is broken. Auto assigns a BugZero manager. Daily KSv2 External Added to denote the issue can be worked on by a contributor
Projects
Status: Done
Development

No branches or pull requests

7 participants