-
Notifications
You must be signed in to change notification settings - Fork 3k
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 2023-02-01] mWeb - Connect Bank account - User can't procced with Chase credentials for special scenario #12123
Comments
Triggered auto assignment to @PauloGasparSv ( |
Triggered auto assignment to @zanyrenney ( |
Please attached the link to TR for special scenario https://expensify.testrail.io/index.php?/cases/view/1971142 |
Having some trouble with VBA locally after clearing my db (want to try this locally), will debug this alongside another VBA issue tomorrow morning! |
@PauloGasparSv I don't know why i have been assigned here, I don't think I should be! I am removing my assignment as this has got engineering assignment. |
Still couldn't figure out this problem and didn't have time to check it today : / , will ask team for help tomorrow if I keep stuck! |
@PauloGasparSv Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@PauloGasparSv Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Triggered auto assignment to @neil-marcellini ( |
Not being redirected back to the same screen but getting a success message instead and being redirected to the routing number page (first page of the "connect manually" flow) Checking locally if that's intended right now. Screen.Recording.2022-10-31.at.16.17.05.mov |
Sorry @neil-marcellini, I think I missclicked the demolition label! |
@kbecciv do you mind re-testing this so I can have a new request to take a look at LogSearch? My staging failed attempt looks completely different and locally I can get past the "Choose the account" screen. |
@kbecciv just re-tested on my physical device in staging and I did not get the same behavior as the evidence video. WhatsApp.Video.2022-11-03.at.4.30.57.PM.mp4Can you re-test? And if it fails, can you send me the email of the applause.expensifail account you are using and in which vault it exists, cause I couldn't find it here. |
@PauloGasparSv Checking, update you shortly with results |
Re-tested with build 1.2.23.9 and had different behavior this time - it directed to log in page. Screen_Recording_20221104-111939_Chrome.mp4 |
Bump @kbecciv |
@nkuoch Checking now, will update you shortly |
@nkuoch Credentials used: Screen_Recording_20221212-095050_Chrome.1.mp4 |
I'll test this again in my physical device and the Android emulator, but in the iOS Simulator mweb and my Desktop Web I was able to proceed all the way to the routing and account number page with those credentials. Screen.Recording.2022-12-12.at.15.41.27.movbtw: I think the credentials had a missing p and an extra a in the email so here they are updated: |
@PauloGasparSv Sorry, over typed the email, updated it |
@PauloGasparSv I can take over , I'm currently doing a refactoring of the flow and believe it will fix this too. |
@nkuoch Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@nkuoch Eep! 4 days overdue now. Issues have feelings too... |
@nkuoch 8 days overdue is a lot. Should this be a Weekly issue? If so, feel free to change it! |
@nkuoch 10 days overdue. Is anyone even seeing these? Hello? |
This issue has not been updated in over 14 days. @nkuoch eroding to Weekly issue. |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a regression has occurred and you are the assigned CM follow the instructions here. If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future. |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.2.58-4 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue: If no regressions arise, payment will be issued on 2023-02-01. 🎊 After the hold period, please check if any of the following need payment for this issue, and if so check them off after paying:
As a reminder, here are the bonuses/penalties that should be applied for any External issue:
|
If you haven’t already, check out our contributing guidelines for onboarding and email [email protected] to request to join our Slack channel!
Action Performed:
Expected Result:
User is able to proceed with Chase credentials for special scenario
Actual Result:
User can't procced with Chase credentials for special scenario
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.2.19.1
Reproducible in staging?: Yes
Reproducible in production?: n/a
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen_Recording_20221024-183029_Chrome.2.mp4
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: