Repository navigation
[HOLD for payment 2024-03-20] [$250] [TS migration] Migrate 'Report' page to TypeScript #25216
Description
Activity
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
59 remaining items
The solution for this issue has been 🚀 deployed to production 🚀 in version 1.4.51-3 and is now subject to a 7-day regression period 📆. Here is the list of pull requests that resolve this issue:
- [TS migration] Migrate 'Report' page to TypeScript #35566
- Follow-up: fix crash on the report screen #38151
If no regressions arise, payment will be issued on 2024-03-20. 🎊
For reference, here are some details about the assignees on this issue:
- @Ollyws requires payment automatic offer (Reviewer)
- @fabioh8010 does not require payment (Contractor)
- @VickyStash 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:
- [@Ollyws] Please propose regression test steps to ensure the new feature will work correctly on production in further releases.
- [@greg-schroeder] Link the GH issue for creating/updating the regression test once above steps have been agreed upon.
Apologies, I was OOO last week. Taking a look at this now!
I might be eligible as well for reviewing 2nd PR which fixes regression
Sending an offer to @aimane-chnaif and @Ollyws for this job (and both PRs)
Accepted, thanks.
Hey @Ollyws - hmm, my dashboard in upwork still says it's waiting for you to accept 🤔
Can you show me what you see?
Oh, nevermind, I am stupid - I sent two offers to you instead of one to each of you. I will remove this one and send a new one to @aimane-chnaif
Okay, fixed!
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
Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @greg-schroeder