-
Notifications
You must be signed in to change notification settings - Fork 2.9k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[$250] [TS migration] Migrate 'FullscreenLoadingIndicator.js' component to TypeScript #25010
Comments
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! |
Hey! I’m Maciej Dobosz from Software Mansion, an expert agency, and I’d like to work on this issue! |
Triggered auto assignment to @AndrewGable, see https://stackoverflow.com/c/expensify/questions/7972 for more details. |
This issue has not been updated in over 15 days. @AndrewGable 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! |
This issue can be closed soon, as the PR is merged. Am I eligible for payment for reviewing and testing the code? (Reviewer Checklist) |
|
Yes, assigned and payment attached to title |
@AndrewGable Thanks! What steps are necessary to close this, as the PR is reviewed & merged? Does the message by Melvin above mean we have to assign a BZ member? ^ |
Triggered auto assignment to @sonialiap ( |
Bug0 Triage Checklist (Main S/O)
|
@sonialiap - Please pay C+ $250 and then close issue. Thanks! |
@cubuspl42, @AndrewGable, @sonialiap Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
1 similar comment
@cubuspl42, @AndrewGable, @sonialiap Uh oh! This issue is overdue by 2 days. Don't forget to update your issues! |
@cubuspl42, @AndrewGable, @sonialiap Eep! 4 days overdue now. Issues have feelings too... |
@sonialiap Would you give it a look so we can close this? Thanks |
@cubuspl42, @AndrewGable, @sonialiap Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@cubuspl42, @AndrewGable, @sonialiap Eep! 4 days overdue now. Issues have feelings too... |
@AndrewGable @sonialiap Bump? |
Sorry @cubuspl42 - This is blocked on @sonialiap. Hopefully after the holidays we can resolve this one. |
@cubuspl42, @AndrewGable, @sonialiap Huh... This is 4 days overdue. Who can take care of this? |
@cubuspl42, @AndrewGable, @sonialiap Eep! 4 days overdue now. Issues have feelings too... |
@sonialiap Would you process the payment when you have a spare moment, please? It's an old thing, and it's an unnecessary notification noise. Thanks! |
@cubuspl42, @AndrewGable, @sonialiap Whoops! This issue is 2 days overdue. Let's get this updated quick! |
@cubuspl42, @AndrewGable, @sonialiap Huh... This is 4 days overdue. Who can take care of this? |
$250 payment sent ✔️ |
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
The text was updated successfully, but these errors were encountered: