Skip to content

Commit 43ec487

Browse files
committed
fix: correct event response logic
1 parent 91aeb59 commit 43ec487

2 files changed

Lines changed: 10 additions & 5 deletions

File tree

src/event/models.py

Lines changed: 5 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -38,4 +38,8 @@ class EventUpdate(BaseModel):
3838
description: str | None = None
3939
repeat: str | None = None
4040
start_date: datetime.date | None = None
41-
end_date: datetime.date | None = None
41+
end_date: datetime.date | None = None
42+
43+
class EventDelete(BaseModel):
44+
result: bool
45+
eid: int

src/event/urls.py

Lines changed: 5 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -7,7 +7,8 @@
77
Event,
88
EventPublic,
99
EventCreate,
10-
EventUpdate
10+
EventUpdate,
11+
EventDelete
1112
)
1213
from event.tables import EventDB
1314
from utils.shared_models import DetailModel, SuccessResponse
@@ -96,7 +97,7 @@ async def create_event(
9697
@router.delete(
9798
"/{eid}",
9899
description="Delete an event",
99-
response_model=SuccessResponse,
100+
response_model=EventDelete,
100101
responses={
101102
404:{"description": "Event doesn't exist."}
102103
},
@@ -108,12 +109,12 @@ async def delete_event(
108109
eid: int
109110
):
110111
rows_deleted = await event.crud.delete_event(db_session, eid)
111-
112+
112113
if rows_deleted == 0:
113114
raise HTTPException(
114115
status_code=status.HTTP_404_NOT_FOUND,
115116
detail="Event doesn't exist."
116117
)
117118

118119
await db_session.commit()
119-
return SuccessResponse(success=True)
120+
return EventDelete(result=True, eid=eid)

0 commit comments

Comments
 (0)