feat: CRUD ListEvent #14
Loading…
Reference in New Issue
Block a user
No description provided.
Delete Branch "VORKOUT-14"
Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?
feat: CRUD ListEventto WIP: feat: CRUD ListEvent@ -0,0 +94,4 @@
# Здесь предполагается, что all_list_event_adapter.validate_python корректно обрабатывает данные
validated_list_event = all_list_event_adapter.validate_python(events_data)
return AllListEventResponse(list_event=validated_list_event, amount_count=total_count, amount_pages=total_pages)
У тебя в returning type указано
Optional[ListEvent]
, а возвращаешь тыAllListEventResponse
@ -0,0 +142,4 @@
return ListEvent.model_validate(listevents_data)
async def update_listevents_by_id(connection: AsyncConnection, update_values, listevents) -> Optional[ListEvent]:
Тоже returning type не соответствует
@ -0,0 +108,4 @@
if listevents_validation is None:
await create_listevents(connection, listevents, user_validation.id)
user_new = await get_listevents_by_name(connection, listevents.name)
Имя переменной я бы поменял)
@ -0,0 +8,4 @@
class ListEventUpdate(Base):
id: Optional[int] = None
Я еще с пользователями когда делал посмотрел
Нужен ли нам id в схемах update?
Просто он приходит в path параметрах и в body его не бывает
@ -0,0 +28,4 @@
status: EventStatus
class AllListEventResponse(Base):
Тут еще надо добавить
current_page
иlimit
, можешь посмотреть вот тутПо большому счету вроде все ок, осталось только от @cyrussmeat дождаться апрува
И перед мержем надо прогнать команду
make format-api
@ -0,0 +22,4 @@
"""
first_event = page * limit - limit
print(creator_id)
Можно убрать, если надо логировать, то лучше через
logger
@ -0,0 +100,4 @@
async def get_listevents_by_name(connection: AsyncConnection, name: str) -> Optional[ListEvent]:
"""
Получает юзера по login.
Тут, видимо, из другого метода скопировал случайно
@ -0,0 +146,4 @@
async def update_listevents_by_id(connection: AsyncConnection, update_values, listevents):
"""
Вносит изменеия в нужное поле таблицы account_table.
Тоже из другого метода описание
@ -18,2 +18,2 @@
creator_id: Optional[int] = None
created_at: Optional[datetime] = None
# creator_id: Optional[int] = None
# created_at: Optional[datetime] = None
Лишнее удалять, не стоит оставлять комменты
@ -12,2 +12,4 @@
raise HTTPException(status_code=status.HTTP_403_FORBIDDEN, detail="You do not have enough permissions")
return authorize_user
async def db_user_role_validation_for_listevents_by_listevent_id(connection, current_user,current_lisevents_creator_id):
Тут опечатка в
current_listevents_creator_id
WIP: feat: CRUD ListEventto feat: CRUD ListEvent