cal/major-domo-database
1
0
Fork 0
Code Issues 25 Pull Requests 25 Actions Packages Projects Releases Wiki Activity

fix: align CustomCommandCreator.discord_id model with BIGINT column (#78)
All checks were successful
Build Docker Image / build (pull_request) Successful in 2m42s
Details

Browse Source 
Closes #78

Change CharField(max_length=20) to BigIntegerField to match the BIGINT
column created by the migration. Remove the str() workaround in
get_creator_by_discord_id() that was compensating for the type mismatch.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
This commit is contained in:
Cal Corum 2026-03-27 01:03:40 -05:00
parent da679b6d1a
commit a85c2e5468
2 changed files with 2 additions and 2 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

2
app/db_engine.py
View File

@ -2224,7 +2224,7 @@ class Decision(BaseModel):
class CustomCommandCreator(BaseModel): class CustomCommandCreator(BaseModel):
"""Model for custom command creators.""" """Model for custom command creators."""
discord_id = CharField(max_length=20, unique=True) # Discord snowflake ID as string discord_id = BigIntegerField(unique=True)
username = CharField(max_length=32) username = CharField(max_length=32)
display_name = CharField(max_length=32, null=True) display_name = CharField(max_length=32, null=True)
created_at = DateTimeField() created_at = DateTimeField()

2
app/routers_v3/custom_commands.py
View File

@ -175,7 +175,7 @@ def delete_custom_command(command_id: int):
def get_creator_by_discord_id(discord_id: int): def get_creator_by_discord_id(discord_id: int):
"""Get a creator by Discord ID""" """Get a creator by Discord ID"""
creator = CustomCommandCreator.get_or_none( creator = CustomCommandCreator.get_or_none(
CustomCommandCreator.discord_id == str(discord_id) CustomCommandCreator.discord_id == discord_id
) )
if creator: if creator:
return model_to_dict(creator) return model_to_dict(creator)