Skip to content

Commit 92be2de

Browse files
authored
Merge pull request #89261 from Krishna2323/krishna2323/issue/86683-2
chore: remove beta gate for bulk duplicate reports
2 parents bfdc6d2 + 31c5d93 commit 92be2de

3 files changed

Lines changed: 2 additions & 7 deletions

File tree

src/CONST/index.ts

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -945,7 +945,6 @@ const CONST = {
945945
PAY_INVOICE_VIA_EXPENSIFY: 'payInvoiceViaExpensify',
946946
SUGGESTED_FOLLOWUPS: 'suggestedFollowups',
947947
GUSTO: 'gustoNewDot',
948-
BULK_DUPLICATE_REPORT: 'bulkDuplicateReport',
949948
BULK_EDIT: 'bulkEdit',
950949
NEW_MANUAL_EXPENSE_FLOW: 'newManualExpenseFlow',
951950
BULK_SUBMIT_APPROVE_PAY: 'bulkSubmitApprovePay',

src/hooks/useSearchBulkActions.ts

Lines changed: 1 addition & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -900,10 +900,6 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
900900
}, []);
901901

902902
const isDuplicateReportOptionVisible = useMemo(() => {
903-
if (!isBetaEnabled(CONST.BETAS.BULK_DUPLICATE_REPORT)) {
904-
return false;
905-
}
906-
907903
if (!isExpenseReportType || !defaultExpensePolicy || selectedReports.length === 0) {
908904
return false;
909905
}
@@ -914,7 +910,7 @@ function useSearchBulkActions({queryJSON}: UseSearchBulkActionsParams) {
914910
}
915911
return report.ownerAccountID === accountID && report.type === CONST.REPORT.TYPE.EXPENSE;
916912
});
917-
}, [isExpenseReportType, defaultExpensePolicy, selectedReports, accountID, isBetaEnabled]);
913+
}, [isExpenseReportType, defaultExpensePolicy, selectedReports, accountID]);
918914

919915
const headerButtonsOptions = useMemo(() => {
920916
if (selectedTransactionsKeys.length === 0 || status == null || !hash) {

tests/unit/hooks/useSearchBulkActionsDuplicateTest.ts

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -106,7 +106,7 @@ jest.mock('@hooks/useConfirmModal', () => ({
106106

107107
jest.mock('@hooks/usePermissions', () => ({
108108
__esModule: true,
109-
default: () => ({isBetaEnabled: (beta: string) => beta === 'bulkDuplicateReport'}),
109+
default: () => ({isBetaEnabled: () => false}),
110110
}));
111111

112112
jest.mock('@hooks/useSelfDMReport', () => ({

0 commit comments

Comments
 (0)