Sidebar: do not show deleted channels

ui/component/sideNavigation/index.js

@@ -18,7 +18,7 @@ import { selectUserVerifiedEmail, selectUser } from 'redux/selectors/user';
 import { selectClientSettings, selectHomepageData } from 'redux/selectors/settings';
 import { doOpenModal, doSignOut } from 'redux/actions/app';
 import { selectUnseenNotificationCount } from 'redux/selectors/notifications';
-import { selectClaimsByUri, selectPurchaseUriSuccess } from 'redux/selectors/claims';
+import { selectClaimsByUri, selectPurchaseUriSuccess, selectResolvingUris } from 'redux/selectors/claims';
 import { selectUserHasValidOdyseeMembership } from 'redux/selectors/memberships';
 import { GetLinksData } from 'util/buildHomepage';
 
@@ -60,6 +60,8 @@ const selectSidebarCategories = createSelector(
 
 function doGetDisplayedSubs(filter) {
   return async (dispatch, getState) => {
+    await waitIfResolving(1000, 5, getState);
+
     const state = getState();
     const claimsByUri = selectClaimsByUri(state);
     const subs = selectSubscriptions(state);
@@ -79,13 +81,22 @@ function doGetDisplayedSubs(filter) {
         });
       } else {
         filteredSubs = lastActiveSubs?.length > 0 ? lastActiveSubs : subs.slice(0, SIDEBAR_SUBS_DISPLAYED);
+        filteredSubs = filteredSubs.filter((sub) => claimsByUri[sub?.uri]);
       }
     }
 
     return filteredSubs;
   };
 }
 
+async function waitIfResolving(waitMs, maxAttempts, getState) {
+  let isResolvingUris = selectResolvingUris(getState()).length > 0;
+  for (let waitCount = 0; isResolvingUris && waitCount < maxAttempts; ++waitCount) {
+    await new Promise((resolve) => setTimeout(resolve, waitMs));
+    isResolvingUris = selectResolvingUris(getState()).length > 0;
+  }
+}
+
 // ****************************************************************************
 // SideNavigation
 // ****************************************************************************

ui/component/sideNavigation/view.jsx

@@ -549,6 +549,7 @@ function SideNavigation(props: Props) {
 
   React.useEffect(() => {
     if (showSubsSection) {
+      // Done this way to avoid over-render from claimsByUris[].
       doGetDisplayedSubs(subscriptionFilter).then((result) => setDisplayedSubs(result));
     }
   }, [subscriptionFilter, showSubsSection, doGetDisplayedSubs]);