-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$500] IOU Report - User is sent to blinking/flashing screen after opening sent IOR request #30282
Comments
Triggered auto assignment to @greg-schroeder ( |
Bug0 Triage Checklist (Main S/O)
|
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open
|
Triggered auto assignment to @AndrewGable ( |
Can you reproduce @greg-schroeder ? |
I think @danielrvidal says he is seeing this in production, can you confirm @lanitochka17 ? |
Demoting to normal bug as this is happening on production according to this! |
Job added to Upwork: https://www.upwork.com/jobs/~0134567ff2a3f3bab0 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @hoangzinh ( |
ProposalPlease re-state the problem that we are trying to solve in this issue.The application blinks when clicking report on the LHN, the indicator also turns red. What is the root cause of that problem?The root cause seems to be on the side of OnyxData, when running the app in localhost it works as expected. The error body from the HTTP response:
What changes do you think we should make in order to solve the problem?This could be a certificate or policy issue on the backend endpoint not allowing access from the staging/production. Without having access to the backend an actual fix for this will be hard to implement. What alternative solutions did you explore? (Optional) |
Here's the issue about retry logic and I think it's being handled here: |
@hungvu193 are you suggesting we hold for the linked issue? |
It looks like a BE issue, after relogin, when userB opens the IOU report, BE returns the error "Policy does not exist or you do not have access. Try refreshing the page." for the API command OpenReport The issue that @hungvu193 mentioned #30282 (comment) is another issue. It won't solve the root cause of this issue. |
@AndrewGable, @hoangzinh, @greg-schroeder Huh... This is 4 days overdue. Who can take care of this? |
Current assignee @hoangzinh is eligible for the Internal assigner, not assigning anyone new. |
Marking internal for now! |
@AndrewGable @hoangzinh @greg-schroeder this issue was created 2 weeks ago. Are we close to a solution? Let's make sure we're treating this as a top priority. Don't hesitate to create a thread in #expensify-open-source to align faster in real time. Thanks! |
This is internal now as it's a BE issue, Expensify engineer will work on shortly |
Same as above |
@AndrewGable @hoangzinh @greg-schroeder this issue is now 3 weeks old. There is one more week left before this issue breaks WAQ. What needs to happen to get a PR in review this week? Please create a thread in #expensify-open-source to discuss. Thanks! |
just wanna confirm @AndrewGable will work on this issue, won't he? |
Yeah @AndrewGable do you mind giving an ETA on this one? |
I'll look at it soon-ish, I want to make sure it still happens before digging into too deep. |
Issue not reproducible during KI retests. (First week) |
@AndrewGable @hoangzinh @greg-schroeder this issue is now 4 weeks old and preventing us from maintaining WAQ. This should now be your highest priority. Please post below what your plan is to get a PR in review ASAP. Thanks! |
@greg-schroeder @AndrewGable it looks like this issue has been fixed somewhere Screen.Recording.2023-11-21.at.21.58.39.mov |
Ah, I think you're right. |
I'm going to close, then. |
If you haven’t already, check out our contributing guidelines for onboarding and email contributors@expensify.com to request to join our Slack channel!
Version Number: 1.3.90-1
Reproducible in staging?: Yes
Reproducible in production?: No
If this was caught during regression testing, add the test name, ID and link from TestRail:
Email or phone of affected tester (no customers):
Logs: https://stackoverflow.com/c/expensify/questions/4856
Expensify/Expensify Issue URL:
Issue reported by: Applause - Internal Team
Slack conversation:
Action Performed:
Expected Result:
User is able to open request and view IOU details
Actual Result:
Blinking page is shown after clicking on report
"Hmm, nothing is here" occurs on occasion instead of blinking page or red dot in LHN
Workaround:
Unknown
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
Android: Native
Bug6249105_1698167408004.XRecorder_24102023_121107.mp4
Android: mWeb Chrome
Bug6249105_1698167328582.RPReplay_Final1698164257.mp4
iOS: Native
iOS: mWeb Safari
MacOS: Chrome / Safari
Bug6249105_1698167222525.Video_2023_10_24-1.mp4
MacOS: Desktop
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: