Refresh notifications on screen open (close #214)

bluesky-social · Feb 16, 2023 · 9359553 · 9359553
1 parent 62077a8
commit 9359553
Show file tree

Hide file tree

Showing 2 changed files with 13 additions and 13 deletions.
diff --git a/src/view/com/notifications/Feed.tsx b/src/view/com/notifications/Feed.tsx
@@ -38,19 +38,19 @@ export const Feed = observer(function Feed({
     }
     return <FeedItem item={item} />
   }
-  const onRefresh = () => {
-    view
-      .refresh()
-      .catch(err =>
-        view.rootStore.log.error('Failed to refresh notifications feed', err),
-      )
+  const onRefresh = async () => {
+    try {
+      await view.refresh()
+    } catch (err) {
+      view.rootStore.log.error('Failed to refresh notifications feed', err)
+    }
   }
-  const onEndReached = () => {
-    view
-      .loadMore()
-      .catch(err =>
-        view.rootStore.log.error('Failed to load more notifications', err),
-      )
+  const onEndReached = async () => {
+    try {
+      await view.loadMore()
+    } catch (err) {
+      view.rootStore.log.error('Failed to load more notifications', err)
+    }
   }
   let data
   if (view.hasLoaded) {

diff --git a/src/view/screens/Notifications.tsx b/src/view/screens/Notifications.tsx
@@ -27,7 +27,7 @@ export const Notifications = ({navIdx, visible}: ScreenParams) => {
       return cleanup
     }
     store.log.debug('Updating notifications feed')
-    store.me.notifications.update().then(() => {
+    store.me.notifications.refresh().then(() => {
       store.me.notifications.updateReadState()
     })
     screen('Notifications')