App
App copied to clipboard
[TS migration] Migrate 'Jest' files 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 |
---|---|
jest/setupMockImages.js | 3 |
jest/setup.js | 5 |
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.
Updated List
Files
Path | Dependencies |
---|---|
jest/setupMockImages.js | 3 |
jest/setupAfterEnv.js | 0 |
jest/setup.js | 7 |
Hi, I'm Julian from Callstack - expert contributor group - and I would like to take a look at this issue.
dibs
@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.
Triggered auto assignment to @zanyrenney (NewFeature
), see https://stackoverflowteams.com/c/expensify/questions/14418#:~:text=BugZero%20process%20steps%20for%20feature%20requests for more details.
Triggered auto assignment to @Beamanator, see https://stackoverflow.com/c/expensify/questions/7972 for more details.
Reviewing
label has been removed, please complete the "BugZero Checklist".
The solution for this issue has been :rocket: deployed to production :rocket: in version 1.4.39-8 and is now subject to a 7-day regression period :calendar:. Here is the list of pull requests that resolve this issue:
- https://github.com/Expensify/App/pull/35316
If no regressions arise, payment will be issued on 2024-02-19. :confetti_ball:
For reference, here are some details about the assignees on this issue:
- @JKobrynski does not require payment (Contractor)
BugZero Checklist: The PR adding this new feature has been merged! The following checklist (instructions) will need to be completed before the issue can be closed:
- [ ] [@JKobrynski] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
- [ ] [@zanyrenney] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.
Skipping the payment summary for this issue since all the assignees are employees or vendors. If this is incorrect, please manually add the payment summary SO.
Thanks Melv. As is correct, closing!
Hey @zanyrenney , I am sending money request on new dot for reviewing https://github.com/Expensify/App/pull/35316. Can you please add a payment summary here? It's $250 for TS migration issues.
bump @zanyrenney ^
payment summary: @thesahindia is due $250 via ND for work on this issue. @JKobrynski is part of callstack, paid outside ND/UW.
$250 approved for @thesahindia