Skip to content

Commit

Permalink
Merge pull request #35563 from Expensify/rodrigo-fix-pin-unresponsive
Browse files Browse the repository at this point in the history
[CP Staging] Fixing some buttons being unresponsive in ReportScreen
  • Loading branch information
rlinoz authored Feb 1, 2024
2 parents 33463c2 + 16f0909 commit 31948fb
Showing 1 changed file with 8 additions and 1 deletion.
9 changes: 8 additions & 1 deletion src/pages/home/ReportScreen.js
Original file line number Diff line number Diff line change
Expand Up @@ -200,6 +200,8 @@ function ReportScreen({
iouReportID: reportProp.iouReportID,
isOwnPolicyExpenseChat: reportProp.isOwnPolicyExpenseChat,
notificationPreference: reportProp.notificationPreference,
isPinned: reportProp.isPinned,
chatReportID: reportProp.chatReportID,
}),
[
reportProp.lastReadTime,
Expand Down Expand Up @@ -230,6 +232,8 @@ function ReportScreen({
reportProp.iouReportID,
reportProp.isOwnPolicyExpenseChat,
reportProp.notificationPreference,
reportProp.isPinned,
reportProp.chatReportID,
],
);

Expand Down Expand Up @@ -658,6 +662,9 @@ export default compose(
prevProps.report.isOptimisticReport === nextProps.report.isOptimisticReport &&
prevProps.report.statusNum === nextProps.report.statusNum &&
_.isEqual(prevProps.report.pendingFields, nextProps.report.pendingFields) &&
prevProps.currentReportID === nextProps.currentReportID,
prevProps.currentReportID === nextProps.currentReportID &&
prevProps.report.notificationPreference === nextProps.report.notificationPreference &&
prevProps.report.isPinned === nextProps.report.isPinned &&
prevProps.report.chatReportID === nextProps.report.chatReportID,
),
);

0 comments on commit 31948fb

Please sign in to comment.