-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
[Payment due Oct 18]Payment issue for E/App PR#49811 #50413
Comments
Triggered auto assignment to @Christinadobrzyn ( |
Looks like PR is under QA review - #49811 I'm going to set this to weekly since it still needs to be in production for 7 days. Payment summaryPayouts due:
|
This was deployed yesterday and we should update the payout date accordingly. |
Payment date to be set in 3 days. (10/17) |
Thanks @mananjadhav! I updated the title to include the payment due date. Do we need a regression test for this? |
No regression test needed. We're just putting the feature behind beta. Hence no offending PR as well. |
Thanks @mananjadhav We'll pay out based on this payment summary on 10/17 - #50413 (comment) |
looks like the PR merged on the 11th of Oct so the 7th day will be 10/18. so we'll pay this out tomorrow! |
Okay, payment day - @mananjadhav, can you please request payment through NewDot? Here's the payment summary |
Done. Thanks @Christinadobrzyn. I think we can close this one out. |
$250 approved for @mananjadhav |
Issue created to compensate the Contributor+ member for their work on #49811
E/E issue linked to the PR - https://www.github.com/Expensify/Expensify/issues/431466.
Contributor+ member who reviewed the PR - @mananjadhav.
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: