gitea-admin/gh-christianlouis-inboxconverge
1
0
Fork 0
Code Issues Pull Requests Actions Packages Projects Releases Wiki Activity

fix: replace deprecated .dict() with .model_dump() and fix db.add() AsyncMock warnings

Browse Source 
Agent-Logs-Url: https://github.com/christianlouis/InboxConverge/sessions/ac39d3f7-d4c3-4417-8c33-8e6340a27281

Co-authored-by: christianlouis <361235+christianlouis@users.noreply.github.com>
This commit is contained in:
copilot-swe-agent[bot]
2026-04-07 10:38:27 +00:00
committed by GitHub
parent 025684386b
commit 31dc059274
6 changed files with 15 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files
backend/app/api/v1/endpoints/admin.py
+2 -2
View File
@@ -322,7 +322,7 @@ async def create_admin_notification_config(
db: AsyncSession = Depends(get_db),
):
"""Create a new admin notification configuration (admin only)"""
config = AdminNotificationConfig(**config_in.dict())
config = AdminNotificationConfig(**config_in.model_dump())
db.add(config)
await db.commit()
await db.refresh(config)
@@ -370,7 +370,7 @@ async def update_admin_notification_config(
detail="Admin notification config not found",
)
update_data = config_in.dict(exclude_unset=True)
update_data = config_in.model_dump(exclude_unset=True)
for field, value in update_data.items():
setattr(config, field, value)
backend/app/api/v1/endpoints/notifications.py
+2 -2
View File
@@ -29,7 +29,7 @@ async def create_notification_config(
db: AsyncSession = Depends(get_db),
):
"""Create a new notification configuration"""
config = NotificationConfig(user_id=current_user.id, **config_in.dict())
config = NotificationConfig(user_id=current_user.id, **config_in.model_dump())
db.add(config)
await db.commit()
await db.refresh(config)
@@ -91,7 +91,7 @@ async def update_notification_config(
detail="Notification config not found",
)
update_data = config_in.dict(exclude_unset=True)
update_data = config_in.model_dump(exclude_unset=True)
for field, value in update_data.items():
setattr(config, field, value)