Skip to content

Commit

Permalink
Merge pull request #36279 from Expensify/arosiclair-should-show-push-…
Browse files Browse the repository at this point in the history
…notification

Check for new shouldShowPushNotification flag
  • Loading branch information
arosiclair authored Feb 12, 2024
2 parents 38b3c59 + 58ca249 commit 29a79da
Show file tree
Hide file tree
Showing 2 changed files with 5 additions and 2 deletions.
2 changes: 1 addition & 1 deletion src/libs/actions/User.ts
Original file line number Diff line number Diff line change
Expand Up @@ -458,7 +458,7 @@ function isBlockedFromConcierge(blockedFromConciergeNVP: OnyxEntry<BlockedFromCo

function triggerNotifications(onyxUpdates: OnyxServerUpdate[]) {
onyxUpdates.forEach((update) => {
if (!update.shouldNotify) {
if (!update.shouldNotify && !update.shouldShowPushNotification) {
return;
}

Expand Down
5 changes: 4 additions & 1 deletion src/types/onyx/OnyxUpdatesFromServer.ts
Original file line number Diff line number Diff line change
Expand Up @@ -2,7 +2,10 @@ import type {OnyxUpdate} from 'react-native-onyx';
import type Request from './Request';
import type Response from './Response';

type OnyxServerUpdate = OnyxUpdate & {shouldNotify?: boolean};
type OnyxServerUpdate = OnyxUpdate & {
shouldNotify?: boolean;
shouldShowPushNotification?: boolean;
};

type OnyxUpdateEvent = {
eventType: string;
Expand Down

0 comments on commit 29a79da

Please sign in to comment.