-
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
Web - Wrong URL when Paypal.me payments page is closed @aneequeahmad #10991
Comments
Triggered auto assignment to @tjferriss ( |
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results. If a production regression has occurred a Root Cause Analysis is required. Please follow the instructions here. 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. |
More info #10939 (comment) |
I was able to reproduce the first part where the wrong URL appeared after closing the Invite member page. I was not able to reproduce the issue with the Paypal page. As mentioned above it looks like there might be similar issues with different pages #10939 (comment). |
Triggered auto assignment to @MonilBhavsar ( |
I'm able to reproduce the issue, but looks like a duplicate issue. @parasharrajat could you please confirm. Thank you! |
Yes, it should be tracked on #11111. |
Thank you! Then I think we can close this issue and other duplicate issues as well and keep only one tracking issue open |
Going to close this issue in favor of #10939, which @parasharrajat has highlighted as having the same root cause. |
Thanks @JmillsExpensify. Please add this to #11111 |
Sorry, what does "add this to $11111" mean? |
You can follow the same pattern and add the issue title and Steps so that we can track it. |
Ok cool, same page. One concern though is that we need to keep track of the bug reporter. It looks like we're fine for the time being, in that I can add @aneequeahmad to the master issue since now reporter is currently listed there. |
Yeah that is fine |
Thank you! I think it is fair to bump to this to weekly then |
Still working on the testing issues. |
Still trying to make time. Low priority. |
Same |
Still on hold |
We're deprecating Paypal, so I'm going to issue payment for reporting and then we can close this issue. |
@aneequeahmad I send you a report offer for this Upwork job: https://www.upwork.com/jobs/~01832a474eb99777e9. Let me know when you've accepted. |
@JmillsExpensify i have accepted the offer. Thank you for inviting |
@JmillsExpensify any update on this issue ? Lets close this one. |
We're closing this issue as Paypal is being deprecated. Which other issue is being referred here? |
@MonilBhavsar im talking about other navigation issues that are listed in navigation project. |
@JmillsExpensify i accepted the offer its been 2 weeks. Please accept it and lets close this offer. Thanks |
Yes please test them all and confirm if they are still reproducible or not. |
Paid out. |
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:
Url should be r/reportId
Actual Result:
Url is /settings/payments
Workaround:
Unknown
Platform:
Where is this issue occurring?
Version Number: 1.2.0.0
Reproducible in staging?: Yes
Reproducible in production?: Yes
Email or phone of affected tester (no customers): any
Logs: https://stackoverflow.com/c/expensify/questions/4856
Notes/Photos/Videos: Any additional supporting documentation
Screen.Recording.2022-08-26.at.3.24.54.AM.1.mov
Expensify/Expensify Issue URL:
Issue reported by: @aneequeahmad
Slack conversation: https://expensify.slack.com/archives/C01GTK53T8Q/p1661466607294799
View all open jobs on GitHub
The text was updated successfully, but these errors were encountered: