mokhtar/shellphone.app
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

memoize isServiceWorkerSupported in useNotifications hook

Browse Source
This commit is contained in:
m5r 2021-09-18 05:35:51 +08:00
parent bb5e664b1e
commit 4d1ef66986
1 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

4
app/core/hooks/use-notifications.ts
View File

@ -1,5 +1,5 @@
import { getConfig, useMutation } from "blitz"; import { getConfig, useMutation } from "blitz";
import { useEffect, useState } from "react"; import { useEffect, useMemo, useState } from "react";
import setNotificationSubscription from "../mutations/set-notification-subscription"; import setNotificationSubscription from "../mutations/set-notification-subscription";
import useCurrentPhoneNumber from "./use-current-phone-number"; import useCurrentPhoneNumber from "./use-current-phone-number";
@ -7,7 +7,7 @@ import useCurrentPhoneNumber from "./use-current-phone-number";
const { publicRuntimeConfig } = getConfig(); const { publicRuntimeConfig } = getConfig();
export default function useNotifications() { export default function useNotifications() {
const isServiceWorkerSupported = "serviceWorker" in navigator; const isServiceWorkerSupported = useMemo(() => "serviceWorker" in navigator, []);
const [subscription, setSubscription] = useState<PushSubscription | null>(null); const [subscription, setSubscription] = useState<PushSubscription | null>(null);
const [setNotificationSubscriptionMutation] = useMutation(setNotificationSubscription); const [setNotificationSubscriptionMutation] = useMutation(setNotificationSubscription);
const phoneNumber = useCurrentPhoneNumber(); const phoneNumber = useCurrentPhoneNumber();