-
Notifications
You must be signed in to change notification settings - Fork 3.6k
Open
Labels
Awaiting PaymentAuto-added when associated PR is deployed to productionAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.Something is broken. Auto assigns a BugZero manager.ExternalAdded to denote the issue can be worked on by a contributorAdded to denote the issue can be worked on by a contributorReviewingHas a PR in reviewHas a PR in reviewWeeklyKSv2KSv2
Description
Coming from #68799 (comment)...
Problem
When every component mounts its own ConfirmModal in the component tree, it creates component bloat, memory overhead from multiple modal instances, and repetitive boilerplate code that makes the application harder to maintain.
Solution
Refactor ConfirmModal usage to useConfirmModal in the following files:
WorkspaceTagsPage.tsx
ImportTagsOptionsPage.tsx
WorkspaceViewTagsPage.tsx
TagSettingsPage.tsx
WorkspaceTaxesPage.tsx
WorkspaceEditTaxPage.tsx
WorkspaceCategoriesPage.tsx
CategorySettingsPage.tsx
WorkspacePerDiemPage.tsx
WorkspacePerDiemDetailsPage.tsx
PolicyDistanceRatesPage.tsx
PolicyDistanceRateDetailsPage.tsx
Issue Owner
Current Issue Owner: @Issue Owner
Current Issue Owner: @NicMendoncaReactions are currently unavailable
Metadata
Metadata
Labels
Awaiting PaymentAuto-added when associated PR is deployed to productionAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.Something is broken. Auto assigns a BugZero manager.ExternalAdded to denote the issue can be worked on by a contributorAdded to denote the issue can be worked on by a contributorReviewingHas a PR in reviewHas a PR in reviewWeeklyKSv2KSv2
Type
Projects
Status
No status