-
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 VideoPlayerPreview
component files to TypeScript
#36725
Comments
Job added to Upwork: https://www.upwork.com/jobs/~01ee4768426d2275b1 |
Triggered auto assignment to Contributor-plus team member for initial proposal review - @ntdiary ( |
Dibs |
Would like to work on it. |
dibs |
I can work on it |
I am Koustubh, a full stack developer. I’d like to work on this job. |
I don't have any TS migration assigned so would like to work on this ticket, this ticket using |
I would like to work on this issue. I'm not being assigned any TS related issues. |
Hello! I'm Kamil Owczarz from Software Mansion, the expert agency and I would like to work on this issue. |
Hi @hayata-suenaga, I just completed my first TS migration for the Expensify project (approved PR here) and am ready to take on another if this is not yet assigned. Please let me know :) |
📣 @ntdiary 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app! |
Hey, I'm leaving the Expensify-SWM team for now and @smelaa will handle this task. |
Hello! I'm Aleksandra Smela from Software Mansion, the expert agency and I would like to work on this issue. |
📣 @smelaa! 📣
|
The BZ member will need to manually hire smelaa for the Contributor role. Please store your Upwork details and apply to our Upwork job so this process is automatic in the future! |
We only need one C+ for the PR so I am removing C+ here to track payments |
📣 @situchan 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app! Offer link |
This issue has not been updated in over 15 days. @hayata-suenaga, @situchan, @smelaa 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! |
The PR was been merged a while ago The only thing left to do is for the payment for the C+ review role for @situchan. @situchan is OOO for April, so I'll changing the priority of this issue to monthly |
What are the next steps here? Should we close this issue? |
I am back |
Triggered auto assignment to @greg-schroeder ( |
@greg-schroeder, please pay @situchan for the C+ review role for this PR |
@greg-schroeder, @hayata-suenaga, @situchan, @smelaa Whoops! This issue is 2 days overdue. Let's get this updated quick! |
Oops, sorry, missed this comment -- looking now |
Paid |
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.
File related to the
VideoPlayerPreview
componentUpwork Automation - Do Not Edit
The text was updated successfully, but these errors were encountered: