-
Notifications
You must be signed in to change notification settings - Fork 3.5k
Description
Context: #62300 (comment)
Let's remove the remaining withOnyx from the project. With that we will be able to remove the HOC from react-native-onyx and remove/optimize a bunch of code that will become useless.
I created some batches to avoid lots of PRs and reviews.
-
Remove remaining withOnyx HOCs from the project | Batch 5 (PR)
-
src/pages/home/report/ReportActionCompose/SuggestionEmoji.tsx
-
src/pages/home/report/ReportActionCompose/SilentCommentUpdater/index.android.tsx
-
src/pages/home/report/ReportActionCompose/SilentCommentUpdater/index.tsx
-
Remove remaining withOnyx HOCs from the project | Batch 7 (PR)
-
src/pages/EnablePayments/AddBankAccount/substeps/ConfirmationStep.tsx
Several attempts were made in the past, so we'll have to revisit this one.
Just cleanup, remove written references to withOnyx, comments, etc. MUST be the last issue to be worked on!
- README.md
- mocks/react-native-onyx.ts
- contributingGuides/STYLE.md
- src/components/TagPicker/index.tsx
- src/libs/ReportUtils.ts
- src/libs/actions/App.ts
- src/pages/home/report/PureReportActionItem.tsx
- src/pages/home/report/ContextMenu/BaseReportActionContextMenu.tsx
- tests/actions/EnforceActionExportRestrictions.ts
cc @roryabraham
Issue Owner
Current Issue Owner: @CortneyOfstadMetadata
Metadata
Labels
Type
Projects
Status