Fix inventory view display issues with campaigns having both timed and sub-based drops

Author: DevilXD

gui.py

@@ -1503,9 +1503,13 @@ def update_progress(self, drop: TimedDrop, label: ttk.Label) -> None:
                     time=drop.ends_at.astimezone().replace(microsecond=0, tzinfo=None)
                 )
         else:
-            progress_text = _("gui", "inventory", "minutes_progress").format(
-                minutes=drop.required_minutes
-            )
+            if drop.required_minutes > 0:
+                progress_text = _("gui", "inventory", "minutes_progress").format(
+                    minutes=drop.required_minutes
+                )
+            else:
+                # required_minutes is zero for subscription-based drops
+                progress_text = ''
             if datetime.now(timezone.utc) < drop.starts_at > drop.campaign.starts_at:
                 # this drop can only be earned later than the campaign start
                 progress_text += '\n' + _("gui", "inventory", "starts").format(

inventory.py

@@ -370,7 +370,7 @@ def has_badge_or_emote(self) -> bool:
 
     @cached_property
     def finished(self) -> bool:
-        return all(d.is_claimed for d in self.drops)
+        return all(d.is_claimed or d.required_minutes <= 0 for d in self.drops)
 
     @cached_property
     def claimed_drops(self) -> int: