Skip to content
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 issue] [TS migration] Migrate MoneyTemporaryForRefactorRequestParticipantsSe #41989

Closed
tgolen opened this issue May 10, 2024 · 10 comments
Assignees
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2

Comments

@tgolen
Copy link
Contributor

tgolen commented May 10, 2024

…lector.js to MoneyRequestParticipantsSelector.tsx

This issue is to cover the extra work that was done in #41216 @DylanDylann Needs to be paid for a C+ review. It was not covered under the original issue, so things got a little mixed up.

Issue OwnerCurrent Issue Owner: @DylanDylann
@tgolen tgolen added External Added to denote the issue can be worked on by a contributor Engineering Bug Something is broken. Auto assigns a BugZero manager. labels May 10, 2024
@tgolen tgolen self-assigned this May 10, 2024
@melvin-bot melvin-bot bot added the Help Wanted Apply this label when an issue is open to proposals by contributors label May 10, 2024
Copy link

melvin-bot bot commented May 10, 2024

Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha (External)

Copy link

melvin-bot bot commented May 10, 2024

Triggered auto assignment to @stephanieelliott (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.

@tgolen tgolen assigned DylanDylann and unassigned getusha May 10, 2024
@melvin-bot melvin-bot bot removed the Help Wanted Apply this label when an issue is open to proposals by contributors label May 10, 2024
@tgolen
Copy link
Contributor Author

tgolen commented May 10, 2024

@stephanieelliott For this one, we just need to pay $250 to @DylanDylann for their C+ review on #41216

Copy link

melvin-bot bot commented May 13, 2024

@tgolen, @stephanieelliott, @DylanDylann Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

@melvin-bot melvin-bot bot added the Overdue label May 13, 2024
Copy link

melvin-bot bot commented May 15, 2024

@tgolen, @stephanieelliott, @DylanDylann Huh... This is 4 days overdue. Who can take care of this?

@stephanieelliott
Copy link
Contributor

stephanieelliott commented May 16, 2024

Sorry for the wait on this, it was assigned just before I went out of office!

Summarizing payment on this issue:

  • Contributor: N/A
  • Contributor+: $250 via Upwork to @DylanDylann - PAID

Upwork job is here: https://www.upwork.com/jobs/~016f4ec9bc2fe4ac85

Copy link

melvin-bot bot commented May 17, 2024

@tgolen, @stephanieelliott, @DylanDylann 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

@stephanieelliott
Copy link
Contributor

Contributor+: $250 via Upwork to @DylanDylann -- I've extended the offer to you, can you please accept when you get a chance? https://www.upwork.com/nx/wm/offer/102342473

Hey @DylanDylann bump on the above! Gonna move this to Weekly so that it doesn't continuously bump the other assignees by going overdue, but please leave a comment here once you have accepted the offer so I can issue your payment.

@stephanieelliott stephanieelliott added Weekly KSv2 and removed Daily KSv2 labels May 17, 2024
@melvin-bot melvin-bot bot removed the Overdue label May 17, 2024
@stephanieelliott stephanieelliott changed the title [TS migration] Migrate MoneyTemporaryForRefactorRequestParticipantsSe [Payment issue] [TS migration] Migrate MoneyTemporaryForRefactorRequestParticipantsSe May 17, 2024
@DylanDylann
Copy link
Contributor

@stephanieelliott I accepted the offer

@stephanieelliott
Copy link
Contributor

Thanks, all paid!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something is broken. Auto assigns a BugZero manager. Engineering External Added to denote the issue can be worked on by a contributor Weekly KSv2
Projects
None yet
Development

No branches or pull requests

4 participants