-
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 2023-08-23] Payment issue for E/App PR#21551 #24317
Comments
Triggered auto assignment to @puneetlath ( |
Job added to Upwork: https://www.upwork.com/jobs/~011a68a4bc16656fb2 |
Bug0 Triage Checklist (Main S/O)
|
Current assignee @allroundexperts is eligible for the Internal assigner, not assigning anyone new. |
Setting to weekly since this isn't on prod yet. |
Payment summary:
@allroundexperts please go ahead and request via NewDot and ping me here when you have. |
This might potentially have a regression (#16805) so I want to wait for things to finalize. |
@allroundexperts did that end up being a regression of this PR? |
Yep. Seems like there was a regression. I'll request half the original amount. Can you please update the payment summary? |
This comment was marked as outdated.
This comment was marked as outdated.
Requested @puneetlath! |
Actually according to @Julesssss comment, this did not cause any regression. So this should be eligible for full payment. @puneetlath Apologies for the inconvenience. |
Ok, ignore my latest comment then. @JmillsExpensify this is the proper payment summary. |
Reviewed the details for @allroundexperts. $1,000 approved for payment based on BZ summary. |
Issue created to compensate the Contributor+ member for their work on #21551
E/E issue linked to the PR - no internal issue.
Contributor+ member who reviewed the PR - @allroundexperts.
Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: