Skip to content

Commit 9ecf4c2

Browse files
leshniakclaude
andcommitted
restore connectWithoutView explanation comment in LocalNotification/index.ts
Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
1 parent 54686bb commit 9ecf4c2

1 file changed

Lines changed: 4 additions & 4 deletions

File tree

  • src/libs/Notification/LocalNotification

src/libs/Notification/LocalNotification/index.ts

Lines changed: 4 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -6,8 +6,8 @@ import BrowserNotifications from './BrowserNotifications';
66
import type {LocalNotificationClickHandler, LocalNotificationModifiedExpenseParams, LocalNotificationModule} from './types';
77

88
let allPolicies: OnyxCollection<Policy>;
9-
// This is a temporary subscription until the modified-expense notification chain is fully migrated
10-
// see https://github.com/Expensify/App/issues/66336
9+
// We do not depend on updates on the UI for notifications, so we can use `connectWithoutView` here.
10+
// This is a temporary subscription until the modified-expense notification chain is fully migrated; see https://github.com/Expensify/App/issues/66336
1111
Onyx.connectWithoutView({
1212
key: ONYXKEYS.COLLECTION.POLICY,
1313
waitForCollectionCallback: true,
@@ -17,8 +17,8 @@ Onyx.connectWithoutView({
1717
});
1818

1919
let allPolicyTags: OnyxCollection<PolicyTagLists>;
20-
// This is a temporary subscription until the modified-expense notification chain is fully migrated
21-
// see https://github.com/Expensify/App/issues/66336
20+
// We do not depend on updates on the UI for notifications, so we can use `connectWithoutView` here.
21+
// This is a temporary subscription until the modified-expense notification chain is fully migrated; see https://github.com/Expensify/App/issues/66336
2222
Onyx.connectWithoutView({
2323
key: ONYXKEYS.COLLECTION.POLICY_TAGS,
2424
waitForCollectionCallback: true,

0 commit comments

Comments
 (0)