Merge pull request 'Fixed bug where the default unit was "m" instead of sending None' (#27 ) from default_unit_patch into master

Reviewed-on: #27
Fixed bug where the default unit was "m" instead of sending None
2026-08-31 20:53:48 +02:00 · 2026-08-31 20:48:51 +02:00 · 2026-08-02 13:50:28 +02:00 · 2026-08-02 13:50:09 +02:00 · 2026-08-02 13:41:23 +02:00 · 2026-08-02 13:40:52 +02:00
2 changed files with 12 additions and 3 deletions
@@ -8,8 +8,8 @@
                "kgs",
                "kilogram",
                "kilograms",
-                "kilogramm",
-                "kilogramms",
+                "kilogramme",
+                "kilogrammes",
                "kg.",
                "kgs."
            ]
@@ -25,5 +25,14 @@
                "pounds"
            ]
        },
+        "tonne": {
+            "factor": 1000,
+            "aliases": [
+                "t",
+                "t.",
+                "tonne",
+                "tonnes"
+            ]
+        }
    }
 }
@@ -117,7 +117,7 @@ class FitnessCog(commands.Cog):
                )
                return

-            unit = parts[2] if len(parts) > 2 else "m"
+            unit = parts[2] if len(parts) > 2 else None

            timestamp = int(message.created_at.timestamp())
            username = message.author.name
Author	SHA1	Message	Date
AllfatherHatt	5c018ab81b	Merge pull request 'Fixed bug where the default unit was "m" instead of sending None' (#27 ) from default_unit_patch into master Reviewed-on: #27	2026-08-31 20:53:48 +02:00
T-BENZIN	d63d03aead	Fixed bug where the default unit was "m" instead of sending None When the log_activity function receives a value of None for the unit parameter, it logs value without applying a conversion multiplier.	2026-08-31 20:48:51 +02:00
T-BENZIN	fffedeccbd	Merge pull request 'Fixed typo in aliases key for tonnes' (#26 ) from tonnes-patch-1 into master Reviewed-on: #26	2026-08-02 13:50:28 +02:00
T-BENZIN	b7bf9494aa	Fixed typo in aliases key for tonnes	2026-08-02 13:50:09 +02:00
T-BENZIN	dbba4533c3	Merge pull request 'added tonnes, fixed ye-olde kilogramme spelling' (#25 ) from lift-tonnes into master Reviewed-on: #25	2026-08-02 13:41:23 +02:00
T-BENZIN	e7c0d8485d	added tonnes, fixed ye-olde kilogramme spelling	2026-08-02 13:40:52 +02:00