-
Notifications
You must be signed in to change notification settings - Fork 2.9k
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
[$250] [TS migration] Migrate 'ReportReactionList' component to TypeScript #31976
Comments
This issue has not been updated in over 15 days. 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! |
Job added to Upwork: https://www.upwork.com/jobs/~011c880c9a39894d4a |
Triggered auto assignment to @isabelastisser ( |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @getusha ( |
I'd like to work on it. |
2 similar comments
I'd like to work on it. |
I'd like to work on it. |
Hi. I've been offline for a while and I would like to work on this. I think it would help me make a good comeback easier. |
I'd like to work on it. |
Upwork job price has been updated to $250 |
I'd like to work on this. |
I'd like to work on it. |
📣 @vadymbokatov 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
Hi. I accepted the offer on Upwork and I'm starting my work on this. Will get the PR ready ASAP. |
Not overdue. |
This issue has not been updated in over 15 days. @isabelastisser, @MonilBhavsar, @getusha, @vadymbokatov 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! |
Reviewing the PR #34549 |
Hi @MonilBhavsar |
Bump @MonilBhavsar for PR review. Thanks! |
Yup, got assigned a review yesterday. It's on my list for today. Got a bit stretched by bugs, but will try to get to it before I end |
PR is merged |
Thanks @MonilBhavsar for your effort on the review. |
Hi, This issue was merged yesterday. Can we please mark it as done and close it, as well as releasing the payment on Upwork? |
I believe it is going to be paid after regression period ends (7 days after PR is deployed to production). I would like @isabelastisser to confirm |
That's correct, @MonilBhavsar! |
Hi. It's been a week since this issue was deployed to production. (based on this comment) |
I will review this today. |
The payments were processed in Upwork, all set. |
TypeScript migration
Make sure you read through our TypeScript's style guide, cheatsheet and PropTypes conversion table before you start working on this migration issue.
Files
Upwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: