Open roryabraham opened 2 weeks ago
Triggered auto assignment to @JmillsExpensify (Bug
), see https://stackoverflow.com/c/expensify/questions/14418 for more details. Please add this bug to a GH project, as outlined in the SO.
Job added to Upwork: https://www.upwork.com/jobs/~021834284536815669432
Triggered auto assignment to Contributor-plus team member for initial proposal review - @shubham1206agra (External
)
Upwork job price has been updated to $75
Migrate from withOnyx HOC to useOnyx hook
withOnyx HOC is deprecated
Whatever props are coming from withOnyx HOC should be replaced with useOnyx hook
Migrate withReportOrNotFound from withOnyx to useOnyx
A new feature
We need to migrate the withOnyx
here to use useOnyx
. We also need to make sure the key we pass to useOnyx
will be the collection member key to prevent the app crashes.
After using useOnyx
, we can remove the type here
Migrate withReportOrNotFound from withOnyx to useOnyx
Migration
We should replace the withOnyx
HOC, shown below, with useOnyx
while cleaning up withReportOrNotFound
.
Going to assign to @Tony-MK since @abhinaybathina and @mkzie2 are assigned to migrate other components. Trying to assign these out first-come-first-serve while spreading them out to as many people as I can
📣 @shubham1206agra 🎉 An offer has been automatically sent to your Upwork account for the Reviewer role 🎉 Thanks for contributing to the Expensify app!
📣 @Tony-MK 🎉 An offer has been automatically sent to your Upwork account for the Contributor role 🎉 Thanks for contributing to the Expensify app!
Offer link Upwork job Please accept the offer and leave a comment on the Github issue letting us know when we can expect a PR to be ready for review 🧑💻 Keep in mind: Code of Conduct | Contributing 📖
@JmillsExpensify, @Tony-MK, @roryabraham, @shubham1206agra Whoops! This issue is 2 days overdue. Let's get this updated quick!
awaiting PR from @Tony-MK
Still waiting for PR
@Tony-MK Bump here
We are going to hold this on https://github.com/Expensify/App/pull/49238 as lint will complain about this.
Coming from https://expensify.slack.com/archives/C01GTK53T8Q/p1725973460005309?thread_ts=1725905735.105989&cid=C01GTK53T8Q
Migrate
src/pages/home/report/withReportOrNotFound.tsx
to useuseOnyx
instead ofwithOnyx
.Upwork Automation - Do Not Edit
Issue Owner
Current Issue Owner: @shubham1206agra