-
Notifications
You must be signed in to change notification settings - Fork 2.8k
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-08-01] [$500] [Report Next Steps Audit] Simplify the report next steps 4 - App optimistic next steps #44784
Comments
Triggered auto assignment to @sakluger ( |
Will look for a C+ to handle this one |
Job added to Upwork: https://www.upwork.com/jobs/~012d18ec8c8471d35f |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @eVoloshchak ( |
Upwork job price has been updated to $500 |
Not looking for proposals, going to pick some volunteer from the C+ team |
Seem like someone took this issue on Slack |
📣 @rayane-djouah 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
The PR has been opened for review 🙂 |
PR is going to be merged soon, we will have some small follow-ups I think |
|
The solution for this issue has been 🚀 deployed to production 🚀 in version 9.0.11-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-08-01. 🎊 For reference, here are some details about the assignees on this issue:
|
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
|
Summarizing payment on this issue: Contributor: @rayane-djouah $500, paid via Upwork |
@eVoloshchak @rayane-djouah do we need to add regression steps for this new feature? |
Based on the design doc here, I think @dylanexpensify will update the test cases to match the new format |
Yep I think that's right! |
I think since regression tests are being handled elsewhere we're good to issue payment on this one right? |
$500 approve for @eVoloshchak |
Yeah I think we can close this one out now that everyone was paid |
Tracking issue
As part of the Update Report Next Steps design doc, implement these new report next steps:
by combining a couple of the existing ones. Please refer to the design document for instructions on how to proceed on the PR.
This needs to be implemented once the backend tasks for this project are done, ideally along the same time to minimize the time when the next steps are showing something else optimistically and when the server response comes in
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @saklugerThe text was updated successfully, but these errors were encountered: