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
2 changed files with 2 additions and 2 deletions
@@ -27,7 +27,7 @@
        },
        "tonne": {
            "factor": 1000,
-            "aliaases": [
+            "aliases": [
                "t",
                "t.",
                "tonne",
@@ -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