fix(token lists): stop showing notifications for lists that are not s…

…elected (#1174)

src/state/lists/updater.ts

@@ -13,6 +13,9 @@ export default function Updater(): null {
   const { library } = useActiveWeb3React()
   const dispatch = useDispatch<AppDispatch>()
   const lists = useSelector<AppState, AppState['lists']['byUrl']>(state => state.lists.byUrl)
+  const selectedListUrl = useSelector<AppState, AppState['lists']['selectedListUrl']>(
+    state => state.lists.selectedListUrl
+  )
 
   const isWindowVisible = useIsWindowVisible()
 
@@ -54,45 +57,49 @@ export default function Updater(): null {
             // automatically update minor/patch as long as bump matches the min update
             if (bump >= min) {
               dispatch(acceptListUpdate(listUrl))
+              if (listUrl === selectedListUrl) {
+                dispatch(
+                  addPopup({
+                    key: listUrl,
+                    content: {
+                      listUpdate: {
+                        listUrl,
+                        oldList: list.current,
+                        newList: list.pendingUpdate,
+                        auto: true
+                      }
+                    }
+                  })
+                )
+              }
+            } else {
+              console.error(
+                `List at url ${listUrl} could not automatically update because the version bump was only PATCH/MINOR while the update had breaking changes and should have been MAJOR`
+              )
+            }
+            break
+
+          case VersionUpgrade.MAJOR:
+            if (listUrl === selectedListUrl) {
               dispatch(
                 addPopup({
                   key: listUrl,
                   content: {
                     listUpdate: {
                       listUrl,
+                      auto: false,
                       oldList: list.current,
-                      newList: list.pendingUpdate,
-                      auto: true
+                      newList: list.pendingUpdate
                     }
-                  }
+                  },
+                  removeAfterMs: null
                 })
               )
-            } else {
-              console.error(
-                `List at url ${listUrl} could not automatically update because the version bump was only PATCH/MINOR while the update had breaking changes and should have been MAJOR`
-              )
             }
-            break
-
-          case VersionUpgrade.MAJOR:
-            dispatch(
-              addPopup({
-                key: listUrl,
-                content: {
-                  listUpdate: {
-                    listUrl,
-                    auto: false,
-                    oldList: list.current,
-                    newList: list.pendingUpdate
-                  }
-                },
-                removeAfterMs: null
-              })
-            )
         }
       }
     })
-  }, [dispatch, lists])
+  }, [dispatch, lists, selectedListUrl])
 
   return null
 }