-
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
[$250] [HOLD for payment 2024-04-09] [HOLD for payment 2024-04-05] [Dev] Infinite loop of GetOlderActions calls #39028
Comments
Triggered auto assignment to @kevinksullivan ( |
sup @kevinksullivan. I already have a WIP PR to fix this here. It's an hourly because it's reproducible only on dev. That means that as soon as we deploy staging it will become a deploy blocker |
👋 Friendly reminder that deploy blockers are time-sensitive ⏱ issues! Check out the open `StagingDeployCash` deploy checklist to see the list of PRs included in this release, then work quickly to do one of the following:
|
Current assignee @roryabraham is eligible for the Engineering assigner, not assigning anyone new. |
ok, jumping back into the PR |
Got that PR ready for review: #39024 |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.57-5 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 2024-04-05. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.58-8 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 2024-04-09. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR fixing this issue has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO. |
Job added to Upwork: https://www.upwork.com/jobs/~017cac9b0408529fb8 |
Current assignee @ishpaul777 is eligible for the External assigner, not assigning anyone new. |
@ishpaul777 offer sent, lmk when you accept! |
hello 👋 @kevinksullivan, i was not involved in the PR no idea why melvin says i am eligible for payment 🤔 |
ah ok, weird haha. Thanks for letting me know. I will close this out 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: only reproducible on dev at the time of creating - 1.4.56-6 at the time of wriging
Reproducible in staging?: no
Reproducible in production?: no
Issue reported by: @ishpaul777
Slack conversation: https://expensify.slack.com/archives/C049HHMV9SM/p1711476371378119?thread_ts=1711147499.521539&cid=C049HHMV9SM
Action Performed:
Expected Result:
GetOlderAction
requests in the network tab.Actual Result:
Skeleton indicator appears and flashes infinitely, along with an infinite loop of
GetOlderActions
requests.Workaround:
n/a
Platforms:
Which of our officially supported platforms is this issue occurring on?
Screenshots/Videos
https://expensify.slack.com/archives/C049HHMV9SM/p1711476420432379?thread_ts=1711147499.521539&cid=C049HHMV9SM
View all open jobs on GitHub
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: