-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Comments
Triggered auto assignment to @stephanieelliott ( |
Job added to Upwork: https://www.upwork.com/jobs/~021884130584906000105 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @brunovjk ( |
This screen: Is part of the old app's code (https://github.com/Expensify/Mobile-Expensify), thus it's not a bug from this code base. |
Not sure if issues for the old app's code are tracked in this repository? |
📣 @brunovjk 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
I'm having a hard time reproducing this issue, I still haven't been able to install "mail app" on my simulator. |
@hannojg have you been able to setup the hybrid app and look into this? |
@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! |
Hey @brunovjk, any luck looking into this yet? |
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. |
Got the hybrid app working and I am able to reproduce! Will look into fixing this now … |
Opened a PR here that fixes the issue: |
Should I review it @hannojg? Thanks |
📣 @ZhenjaHorbach 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
@stephanieelliott |
@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. |
Sorry 😅 But now yes |
No worries! All good 😅, will pay this one out at the end of the regression period |
I think this should've went out with the 2/26 deploy, which would make payment due tomorrow. Queuing for payment on 3/5 |
Summarizing payment on this issue:
Upwork job is here: https://www.upwork.com/jobs/~021884130584906000105 |
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:
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:
Screenshots/Videos
bug.mp4
bug2.mp4
View all open jobs on GitHub
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @stephanieelliottThe text was updated successfully, but these errors were encountered: