-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Payment issue for E/App PR#19675 #24209
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
Comments
Triggered auto assignment to @laurenreidexpensify ( |
Job added to Upwork: https://www.upwork.com/jobs/~012911ed58beaa305b |
Bug0 Triage Checklist (Main S/O)
|
Current assignee @parasharrajat is eligible for the Internal assigner, not assigning anyone new. |
@madmax330 looking at https://github.com/Expensify/Expensify/issues/215650 it's unclear whether the PR has hit prod yet - can you confirm the PR link pls? |
The PR is here: #19675 |
Great thanks |
This has been on prod for a week now #19675 (comment) so payment can be issued |
Sorry @parasharrajat total force of habit - I sent you a job on Upwork :face_palm: |
Payment Summary:
|
@parasharrajat @laurenreidexpensify 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! |
1 similar comment
@parasharrajat @laurenreidexpensify 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! |
@parasharrajat cna you confirm if you've requested payment for this one? |
@laurenreidexpensify Not yet, the original PR was reverted which we still need to redeploy. I don't think this is quite ready for payment yet. |
Commented on the other issue, will try to get this back up by EOW |
Im' gonna remove myself from this issue, cos @parasharrajat can request payment directly in newdot when this is ready, so no need for extra oversight from me 👍 |
@madmax330 Bump. |
PR is back up. Still need to do some testing |
This issue has not been updated in over 15 days. @madmax330, @parasharrajat eroding to Monthly issue. P.S. Is everyone reading this sure this is really a near-term priority? Be brave: if you disagree, go ahead and close it out. If someone disagrees, they'll reopen it, and if they don't: one less thing to do! |
@madmax330, @parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing. If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead. |
@madmax330, @parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing. If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead. |
@madmax330, @parasharrajat, this Monthly task hasn't been acted upon in 6 weeks; closing. If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead. |
Looks like this change was deprioritized and thus PR #30782 (comment) was closed. @laurenreidexpensify Can you please finalize the payment summary now? This #24209 (comment) was the last summary. |
Contributor+: @parasharrajat due $500 via NewDot After much, much back-n-forth with Rajat here, he's being compensated 50% of $1000 for the work he did reviewing PRs that were closed. |
Payment requested as per #24209 (comment) |
$500 approved for @parasharrajat |
Uh oh!
There was an error while loading. Please reload this page.
Issue created to compensate the Contributor+ member for their work on #19675
E/E issue linked to the PR - https://www.github.com/Expensify/Expensify/issues/215650.
Contributor+ member who reviewed the PR - @parasharrajat.
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: