Repository navigation
[HOLD for payment 2024-02-14] [HOLD #24918][TS migration] Migrate 'IOU.js' lib to TypeScript #24926
Description
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/libs/actions/IOU.js | 23 |
Activity
melvin-bot commentedon Sep 11, 2023
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!
VickyStash commentedon Oct 2, 2023
Hi, I'm Viktoryia from Callstack - expert contributor group - and I would like to work on this issue.
roryabraham commentedon Nov 2, 2023
Excited for this one – I think it will be particularly valuable. When can we expect a PR?
VickyStash commentedon Nov 3, 2023
@roryabraham For now, this one is blocked by ReportUtils and Policy.js migrations. I'll prioritize it as soon as it gets unblocked!
35 remaining items
garrettmknight commentedon Feb 14, 2024
Reassigning for payment. Should be a quick one once we decide!
mallenexpensify commentedon Feb 18, 2024
@roryabraham 👀 please on the above.
I'm brand new here as BZ so I don't want to jump in without much context.
garrettmknight commentedon Feb 21, 2024
Taking this one back over - @roryabraham can you comment on the complexity of this one?
garrettmknight commentedon Feb 23, 2024
@roryabraham mind chiming in?
roryabraham commentedon Feb 23, 2024
Yeah, I think it's fair to bump this one up to $750 due to the high volume of testing that had to happen. Saying that because it's probably 3x a "standard" TS migration issue
garrettmknight commentedon Feb 26, 2024
Cool, all paid up. Closing!
melvin-bot commentedon Feb 27, 2024
If you are the assigned CME please investigate whether the linked PR caused a regression and leave a comment with the results.
If a regression has occurred and you are the assigned CM follow the instructions here.
If this regression could have been avoided please consider also proposing a recommendation to the PR checklist so that we can avoid it in the future.