Skip to content

[Due for payment 2026-02-04] Refactor ConfirmModal usage to useConfirmModal in Report and Home pages #76695

Description

@roryabraham

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:

ReportDetailsExportPage.tsx
PureReportActionItem.tsx
PopoverReportActionContextMenu.tsx
HeaderView.tsx
FloatingActionButtonAndPopover.tsx
ReportDetailsPage.tsx
ReportParticipantsPage.tsx
RoomMembersPage.tsx
RoomMemberDetailsPage.tsx
ReportParticipantDetailsPage.tsx
EditReportFieldPage.tsx
TravelTerms.tsx
Issue OwnerCurrent Issue Owner: @
Issue OwnerCurrent Issue Owner: @ZhenjaHorbach

Activity

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Labels

Awaiting PaymentAuto-added when associated PR is deployed to productionBugSomething is broken. Auto assigns a BugZero manager.DailyKSv2ExternalAdded to denote the issue can be worked on by a contributor

Type

No type

Projects

Milestone

No milestone

Relationships

None yet

Development

No branches or pull requests

Issue actions