-
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 2024-05-02] CRITICAL: [Track Expense] [$500] Unable to edit request #38699
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01585d11e9c8f2b823 |
Triggered auto assignment to Contributor Plus for review of internal employee PR - @mollfpr ( |
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 only API that seems to still be failing is UpdateMoneyRequestAmountAndCurrency
|
Fix for the above is merged - pending deploy https://github.com/Expensify/Auth/pull/10407 |
@shubham1206agra Are you able to link the draft PR you have here? |
@thienlnam Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@shubham1206agra Bump on this - the BE issues are resolved so you should be able to open your PR now |
@thienlnam, @shubham1206agra Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@shubham1206agra Bump again - what's blocking us from getting this into review? |
Nothing as for now. I will ready this PR after merging track whisper. |
Increasing amount since we'd like to fit this into the next App deploy |
Is this issue open for Contributors??? I can not find |
@skyweb331 I think @shubham1206agra is already on this |
Or at the very least, I'm confused by their comment here: #38699 (comment) |
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.4.65-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-05-02. 🎊 For reference, here are some details about the assignees on this issue:
|
Issue is ready for payment but no BZ is assigned. @twisterdotcom you are the lucky winner! Please verify the payment summary looks correct and complete the checklist. Thanks! |
Payment Summary
BugZero Checklist (@twisterdotcom)
|
My payment is pending as well here: |
@twisterdotcom My payment is pending too. I have never received any payment. |
@shubham1206agra you still need to accept the offer I sent here: https://www.upwork.com/nx/wm/offer/102145720 |
@twisterdotcom Accepted. |
Nice, okay @allroundexperts I updated the Summary here: #38699 (comment) to include you and paid out @shubham1206agra. |
$500 approved for @allroundexperts |
Things to fix from this PR: #37282
#37282 (review)
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: