Skip to content

Commit

Permalink
Merge pull request #53064 from bernhardoj/fix/not-found-when-linking-…
Browse files Browse the repository at this point in the history
…to-comment

[CP Staging] Fix not found view when linking to a message
  • Loading branch information
mountiny authored Nov 25, 2024
2 parents 604d343 + 095deaf commit cfaec15
Show file tree
Hide file tree
Showing 4 changed files with 12 additions and 1 deletion.
8 changes: 8 additions & 0 deletions src/libs/actions/IOU.ts
Original file line number Diff line number Diff line change
Expand Up @@ -6125,6 +6125,7 @@ function getSendMoneyParams(
key: `${ONYXKEYS.COLLECTION.REPORT}${optimisticTransactionThread.reportID}`,
value: {
participants: redundantParticipants,
isOptimisticReport: false,
},
},
{
Expand All @@ -6141,6 +6142,13 @@ function getSendMoneyParams(
key: `${ONYXKEYS.COLLECTION.TRANSACTION}${optimisticTransaction.transactionID}`,
value: {pendingAction: null},
},
{
onyxMethod: Onyx.METHOD.MERGE,
key: `${ONYXKEYS.COLLECTION.REPORT}${chatReport.reportID}`,
value: {
isOptimisticReport: false,
},
},
{
onyxMethod: Onyx.METHOD.MERGE,
key: `${ONYXKEYS.COLLECTION.REPORT_ACTIONS}${chatReport.reportID}`,
Expand Down
2 changes: 2 additions & 0 deletions src/libs/actions/Policy/Policy.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2311,6 +2311,7 @@ function createWorkspaceFromIOUPayment(iouReport: OnyxEntry<Report>): WorkspaceF
addWorkspaceRoom: null,
},
pendingAction: null,
isOptimisticReport: false,
},
},
{
Expand All @@ -2330,6 +2331,7 @@ function createWorkspaceFromIOUPayment(iouReport: OnyxEntry<Report>): WorkspaceF
addWorkspaceRoom: null,
},
pendingAction: null,
isOptimisticReport: false,
},
},
{
Expand Down
1 change: 1 addition & 0 deletions src/libs/actions/TeachersUnite.ts
Original file line number Diff line number Diff line change
Expand Up @@ -137,6 +137,7 @@ function addSchoolPrincipal(firstName: string, partnerUserID: string, lastName:
pendingFields: {
addWorkspaceRoom: null,
},
isOptimisticReport: false,
pendingAction: null,
},
},
Expand Down
2 changes: 1 addition & 1 deletion src/pages/home/ReportScreen.tsx
Original file line number Diff line number Diff line change
Expand Up @@ -515,7 +515,7 @@ function ReportScreen({route, currentReportID = '', navigation}: ReportScreenPro
// This function is triggered when a user clicks on a link to navigate to a report.
// For each link click, we retrieve the report data again, even though it may already be cached.
// There should be only one openReport execution per page start or navigating
fetchReportIfNeeded();
fetchReport();
// eslint-disable-next-line react-compiler/react-compiler, react-hooks/exhaustive-deps
}, [route, isLinkedMessagePageReady, isLoadingReportOnyx, reportActionIDFromRoute]);

Expand Down

0 comments on commit cfaec15

Please sign in to comment.