feat: update progress handling in format_description and history_modal

Author: FuzzyGrim

src/app/helpers.py

@@ -140,22 +140,23 @@ def format_description(field_name, old_value, new_value, media_type=None):  # no
 
     if field_name == "progress" and media_type != MediaTypes.MOVIE.value:
         diff = new_value - old_value
+        diff_abs = abs(diff)
 
         if media_type == MediaTypes.GAME.value:
             if diff > 0:
-                return f"Added {minutes_to_hhmm(diff)} of playtime"
-            return f"Removed {minutes_to_hhmm(abs(diff))} of playtime"
+                return f"Added {minutes_to_hhmm(diff_abs)} of playtime"
+            return f"Removed {minutes_to_hhmm(diff_abs)} of playtime"
 
         unit = (
             f"{media_type_config.get_unit(media_type, short=False).lower()}"
-            f"{pluralize(diff)}"
+            f"{pluralize(diff_abs)}"
         )
 
         verb = media_type_config.get_verb(media_type, past_tense=True).title()
         if diff < 0:
             verb = "Removed"  # Override with "Removed" for negative changes
 
-        return f"{verb} {abs(diff)} {unit}"
+        return f"{verb} {diff_abs} {unit}"
 
     if field_name == "repeats":
         verb = media_type_config.get_verb(media_type, past_tense=True)

src/app/views.py

@@ -574,6 +574,8 @@ def history_modal(
                 delta = new_record.diff_against(old_record)
 
                 for change in delta.changes:
+                    if change.field == "progress_changed":
+                        continue
                     entry["changes"].append(
                         {
                             "description": helpers.format_description(
@@ -594,7 +596,7 @@ def history_modal(
                 for field in history_model._meta.get_fields():  # noqa: SLF001
                     if (
                         field.name.startswith("history_")
-                        or field.name in ["id", "modified"]
+                        or field.name in ["id", "progress_changed"]
                         or not getattr(new_record, field.attname)
                     ):
                         continue