Expensify / App

Welcome to New Expensify: a complete re-imagination of financial collaboration, centered around chat. Help us build the next generation of Expensify by sharing feedback and contributing to the code.
https://new.expensify.com
MIT License
3.53k stars 2.88k forks source link

[HOLD for payment 2023-11-29] [$250] [TS migration] Migrate 'Modal' component to TypeScript #25139

Closed melvin-bot[bot] closed 11 months ago

melvin-bot[bot] commented 1 year ago

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/components/Modal/modalPropTypes.js 5
src/components/Modal/index.web.js 8
src/components/Modal/index.ios.js 4
src/components/Modal/index.android.js 4
src/components/Modal/BaseModal.js 12
Upwork Automation - Do Not Edit
  • Upwork Job URL: https://www.upwork.com/jobs/~0197dfdc0abef354fc
  • Upwork Job ID: 1722315062733119488
  • Last Price Increase: 2023-11-28
  • Automatic offers:
    • situchan | Reviewer | 27580419
Issue OwnerCurrent Issue Owner: @slafortune
melvin-bot[bot] commented 1 year ago

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!

JKobrynski commented 1 year ago

Hi, I'm Julian from Callstack - expert contributor group - and I would like to take a look at this issue.

JKobrynski commented 1 year ago

The PR is ready for review!

melvin-bot[bot] commented 1 year ago

Job added to Upwork: https://www.upwork.com/jobs/~0197dfdc0abef354fc

melvin-bot[bot] commented 1 year ago

Current assignee @situchan is eligible for the External assigner, not assigning anyone new.

mountiny commented 1 year ago

PR is being reviewed

melvin-bot[bot] commented 1 year ago

📣 @situchan 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!

Offer link Upwork job

melvin-bot[bot] commented 11 months ago

@mountiny, @JKobrynski, @situchan 6 days overdue. This is scarier than being forced to listen to Vogon poetry!

mountiny commented 11 months ago

Has everything been covered for this issue? @JKobrynski ?

melvin-bot[bot] commented 11 months ago

@mountiny, @JKobrynski, @situchan Uh oh! This issue is overdue by 2 days. Don't forget to update your issues!

melvin-bot[bot] commented 11 months ago

Upwork job price has been updated to $250

melvin-bot[bot] commented 11 months ago

Triggered auto assignment to @slafortune (Bug), see https://stackoverflow.com/c/expensify/questions/14418 for more details.

melvin-bot[bot] commented 11 months ago

Bug0 Triage Checklist (Main S/O)

mountiny commented 11 months ago

$250 to @situchan for review and testing please cc @slafortune

slafortune commented 11 months ago

Paid 👍