-
Notifications
You must be signed in to change notification settings - Fork 3.2k
Payment issue for E/App PR#59672 #60840
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 @isabelastisser ( |
@isabelastisser, @ZhenjaHorbach Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
Not overdue |
@ZhenjaHorbach, just checking in to see if this is good for payment. Thanks! |
Bump @ZhenjaHorbach. Is this good for payment? |
Oh @JS00001 |
To be honest, I'm not sure. IDK exactly what the order of events is supposed to be, but Zhenja spent a lot of time reviewing the first PR! |
@luacmartins tagging you here to clarify the payment details since you are assigned to the PR. Thanks! |
I think we should issue payment as usual here since it involved a considerable amount of time to review the first PR which was quite large. |
Thanks, @luacmartins! |
Payment summary: @ZhenjaHorbach $250 for PR -- Pending in NewDot. |
Issue created to compensate the Contributor+ member for their work on #59672
E/E issue linked to the PR - https://www.github.com/Expensify/Expensify/issues/487125.
Contributor+ member who reviewed the PR - @ZhenjaHorbach.
Wait 7 days after the PR is deployed to production before issuing payment in case there are regressions.
The text was updated successfully, but these errors were encountered: