App
App copied to clipboard
[$250] [TS migration] Migrate 'NewTask' page to TypeScript
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
| Path | Dependencies |
|---|---|
| src/pages/tasks/NewTaskTitlePage.js | 18 |
| src/pages/tasks/NewTaskPage.js | 23 |
| src/pages/tasks/NewTaskDetailsPage.js | 18 |
| src/pages/tasks/NewTaskDescriptionPage.js | 18 |
Upwork Automation - Do Not Edit
- Upwork Job URL: https://www.upwork.com/jobs/~011d7e945e3d56a16e
- Upwork Job ID: 1750844014325379072
- Last Price Increase: 2024-01-26
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!
@melvin-bot[bot], this Monthly task hasn't been acted upon in 6 weeks; closing.
If you disagree, feel encouraged to reopen it -- but pick your least important issue to close instead.
Hi, I'm Kacper from Callstack - expert contributor group - and I would like to take a look at this issue.
Job added to Upwork: https://www.upwork.com/jobs/~011d7e945e3d56a16e
Triggered auto assignment to Contributor-plus team member for initial proposal review - @thesahindia (External)
Dibs
Upwork job price has been updated to $250
Triggered auto assignment to @alexpensify (NewFeature), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.
Thank you, everyone, for the energy here, but please follow the contributor guidelines. If you are interested in this one, then please submit a proposal for the changes that are important here. After, it will go through a review process.
Correction to my post from Friday. This one has been assigned to @fvlvte and they will work on a proposal this one.
@fvlvte any update on how the update is going?
No update
Hey, sorry, missed notification I was on another task, ll open draft today @alexpensify
All good, thank you!
Weekly Update: @fvlvte any news here? Thanks!
Hey, I got reassigned to another project so I think somebody from Callstack team will pick it over
Hi, I'm Eugene from Callstack - an expert contributor group Iād like to work on this issue
Hey @alexpensify I just finished another TS migration job and would like to handle this one now. Looks like it's still unassigned but please let me know.
Proposal:
I would review current functionality and determine where these files are imported or otherwise used (e.g. via Routing). Once I understand their use I would amend their filenames to TS or TSX (where applicable) and amend the code in all four files to TS. I would then test them on all platforms and ensure no errors or changes in functionality are taking place prior to submitting PR for review.
I'm catching up from being OOO and assigned @pasyukevich.
I am planning to go OOO. Please reassign.
I asked for C+ help here:
https://expensify.slack.com/archives/C02NK2DQWUX/p1709073195364429
š£ @c3024 š An offer has been automatically sent to your Upwork account for the Contributor role š Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review š§āš» Keep in mind: Code of Conduct | Contributing š
@c3024 will be stepping in as the C+ here. Thanks!
@pasyukevich - Any update here? Thanks!
@alexpensify PR was opened for review, waiting for a second review
Awesome, thanks for the update.
Thanks for the update, it looks like the review happened and we are moving forward here.
Weekly Update: PR is in review
Deployed to production. Automation hasn't triggered here.
This looks like it slipped my payment review process, I need to review this one more tomorrow since we might be past the 7-day mark.