-
Notifications
You must be signed in to change notification settings - Fork 3.2k
[$250] Need a C+ review for https://github.com/Expensify/App/pull/10452 #11067
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 @kevinksullivan for Upwork job creation (chore: |
Triggered auto assignment to @jliexpensify for Upwork job creation (chore: |
Ohhh, a CM needs to create an issue. Thought the assigner was not working properly, sorry about that double assignment. |
Posted. Internal - https://www.upwork.com/ab/applicants/1570937596735205376/job-details |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @Santhosh-Sellavel ( |
Triggered auto assignment to @pecanoro ( |
Hired @Santhosh-Sellavel |
@iwiznia I am not completely sure what we need to do here. I got assigned but I have no clue if you are looking for a CME or C+ to review the PR. |
Sorry, that was the auto assigner, you are not needed here @pecanoro |
Can we keep this open until payment is issued? |
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. |
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. |
According to @luacmartins - it looks like there wasn't actually an issue in #11503? So we're not reducing @Santhosh-Sellavel 's pay here right? |
@jliexpensify that's correct! |
Not overdue. |
@jliexpensify This is due for payment, and deployed to production 12 days ago! |
@Santhosh-Sellavel apologies, just looked at the other issue. Paying now. |
Paid, job closed. |
Creating this issue to trigger auto assigner for a review on #10452
The text was updated successfully, but these errors were encountered: