fix(notifications): avoid over-notifying

Author: kentcdodds

packages/workshop-app/app/routes/admin+/notifications.server.tsx

@@ -22,6 +22,8 @@ const NOTIFICATIONS: Array<Notification> = [
 ]
 
 export async function getUnmutedNotifications() {
+	if (ENV.EPICSHOP_DEPLOYED) return []
+
 	const muted = await getMutedNotifications()
 
 	const visibleNotifications = NOTIFICATIONS.filter((n) => {

packages/workshop-app/app/routes/admin+/notifications.tsx

@@ -26,10 +26,15 @@ export function Notifications({
 }) {
 	const fetcher = useFetcher<typeof action>()
 	const fetcherRef = useRef(fetcher)
+	const toastedIds = useRef<Set<string>>(new Set())
 
 	useEffect(() => {
 		for (const notification of unmutedNotifications) {
+			if (toastedIds.current.has(notification.id)) continue
+			toastedIds.current.add(notification.id)
+
 			toast.info(notification.title, {
+				id: notification.id,
 				description: (
 					<div>
 						<p>{notification.message}</p>