perf: Improved performance by optimizing API calls to only refresh when the tick prompts a refresh. All UI is now significantly faster

src/app/radarr/mod.rs

@@ -154,7 +154,6 @@ impl<'a> App<'a> {
         self.cancellation_token.cancel();
       } else {
         self.dispatch_by_radarr_block(&active_radarr_block).await;
-        self.refresh_radarr_metadata().await;
       }
     }
 

src/app/radarr/radarr_tests.rs

@@ -563,22 +563,6 @@ mod tests {
 
       app.radarr_on_tick(ActiveRadarrBlock::Downloads).await;
 
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        RadarrEvent::GetDownloads.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        RadarrEvent::GetQualityProfiles.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        RadarrEvent::GetTags.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        RadarrEvent::GetRootFolders.into()
-      );
       assert_eq!(
         sync_network_rx.recv().await.unwrap(),
         RadarrEvent::GetDownloads.into()

src/app/sonarr/mod.rs

@@ -18,9 +18,6 @@ impl<'a> App<'a> {
         self
           .dispatch_network_event(SonarrEvent::ListSeries.into())
           .await;
-        self
-          .dispatch_network_event(SonarrEvent::GetDownloads.into())
-          .await;
       }
       ActiveSonarrBlock::SeriesDetails => {
         self.is_loading = true;
@@ -63,6 +60,9 @@ impl<'a> App<'a> {
           .await;
       }
       ActiveSonarrBlock::Blocklist => {
+        self
+          .dispatch_network_event(SonarrEvent::ListSeries.into())
+          .await;
         self
           .dispatch_network_event(SonarrEvent::GetBlocklist.into())
           .await;
@@ -156,7 +156,6 @@ impl<'a> App<'a> {
         self.cancellation_token.cancel();
       } else {
         self.dispatch_by_sonarr_block(&active_sonarr_block).await;
-        self.refresh_sonarr_metadata().await;
       }
     }
 

src/app/sonarr/sonarr_tests.rs

@@ -22,6 +22,10 @@ mod tests {
         .await;
 
       assert!(app.is_loading);
+      assert_eq!(
+        sync_network_rx.recv().await.unwrap(),
+        SonarrEvent::ListSeries.into()
+      );
       assert_eq!(
         sync_network_rx.recv().await.unwrap(),
         SonarrEvent::GetBlocklist.into()
@@ -232,10 +236,6 @@ mod tests {
         sync_network_rx.recv().await.unwrap(),
         SonarrEvent::ListSeries.into()
       );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetDownloads.into()
-      );
       assert!(!app.data.sonarr_data.prompt_confirm);
       assert_eq!(app.tick_count, 0);
     }
@@ -478,26 +478,6 @@ mod tests {
 
       app.sonarr_on_tick(ActiveSonarrBlock::Downloads).await;
 
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetDownloads.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetQualityProfiles.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetLanguageProfiles.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetTags.into()
-      );
-      assert_eq!(
-        sync_network_rx.recv().await.unwrap(),
-        SonarrEvent::GetRootFolders.into()
-      );
       assert_eq!(
         sync_network_rx.recv().await.unwrap(),
         SonarrEvent::GetDownloads.into()