feat: CRUD ListEvent #14

Merged
ivan.dev merged 8 commits from VORKOUT-14 into master 2025-07-04 18:06:03 +05:00
Member
No description provided.
ivan.dev added 1 commit 2025-06-30 15:38:23 +05:00
ivan.dev changed title from feat: CRUD ListEvent to WIP: feat: CRUD ListEvent 2025-06-30 15:38:27 +05:00
ivan.dev requested review from cyrussmeat 2025-06-30 15:38:37 +05:00
ivan.dev requested review from vlad.dev 2025-06-30 15:38:37 +05:00
vlad.dev requested changes 2025-06-30 15:48:40 +05:00
@ -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)
Member

У тебя в returning type указано Optional[ListEvent], а возвращаешь ты AllListEventResponse

У тебя в returning type указано `Optional[ListEvent]`, а возвращаешь ты `AllListEventResponse `
ivan.dev marked this conversation as resolved
@ -0,0 +142,4 @@
return ListEvent.model_validate(listevents_data)
async def update_listevents_by_id(connection: AsyncConnection, update_values, listevents) -> Optional[ListEvent]:
Member

Тоже returning type не соответствует

Тоже returning type не соответствует
ivan.dev marked this conversation as resolved
@ -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)
Member

Имя переменной я бы поменял)

Имя переменной я бы поменял)
ivan.dev marked this conversation as resolved
@ -0,0 +8,4 @@
class ListEventUpdate(Base):
id: Optional[int] = None
Member

Я еще с пользователями когда делал посмотрел
Нужен ли нам id в схемах update?
Просто он приходит в path параметрах и в body его не бывает

Я еще с пользователями когда делал посмотрел Нужен ли нам id в схемах update? Просто он приходит в path параметрах и в body его не бывает
ivan.dev marked this conversation as resolved
@ -0,0 +28,4 @@
status: EventStatus
class AllListEventResponse(Base):
Member

Тут еще надо добавить current_page и limit, можешь посмотреть вот тут

Тут еще надо добавить `current_page` и `limit`, можешь посмотреть вот [тут](https://gitea.heado.ru/Vorkout/connect/src/branch/VORKOUT-8/api/api/db/logic/account.py#L48)
ivan.dev marked this conversation as resolved
ivan.dev added 1 commit 2025-07-01 22:31:19 +05:00
vlad.dev approved these changes 2025-07-01 23:11:56 +05:00
vlad.dev left a comment
Member

По большому счету вроде все ок, осталось только от @cyrussmeat дождаться апрува
И перед мержем надо прогнать команду make format-api

По большому счету вроде все ок, осталось только от @cyrussmeat дождаться апрува И перед мержем надо прогнать команду `make format-api`
@ -0,0 +22,4 @@
"""
first_event = page * limit - limit
print(creator_id)
Member

Можно убрать, если надо логировать, то лучше через logger

Можно убрать, если надо логировать, то лучше через `logger`
ivan.dev marked this conversation as resolved
@ -0,0 +100,4 @@
async def get_listevents_by_name(connection: AsyncConnection, name: str) -> Optional[ListEvent]:
"""
Получает юзера по login.
Member

Тут, видимо, из другого метода скопировал случайно

Тут, видимо, из другого метода скопировал случайно
ivan.dev marked this conversation as resolved
@ -0,0 +146,4 @@
async def update_listevents_by_id(connection: AsyncConnection, update_values, listevents):
"""
Вносит изменеия в нужное поле таблицы account_table.
Member

Тоже из другого метода описание

Тоже из другого метода описание
ivan.dev marked this conversation as resolved
ivan.dev added 1 commit 2025-07-02 11:37:11 +05:00
ivan.dev added 1 commit 2025-07-02 11:43:27 +05:00
ivan.dev added 1 commit 2025-07-02 11:48:26 +05:00
mikhail.dev reviewed 2025-07-02 14:13:20 +05:00
@ -18,2 +18,2 @@
creator_id: Optional[int] = None
created_at: Optional[datetime] = None
# creator_id: Optional[int] = None
# created_at: Optional[datetime] = None
First-time contributor

Лишнее удалять, не стоит оставлять комменты

Лишнее удалять, не стоит оставлять комменты
ivan.dev marked this conversation as resolved
vlad.dev added 1 commit 2025-07-02 14:14:09 +05:00
vlad.dev reviewed 2025-07-02 14:36:42 +05:00
@ -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):
Member

Тут опечатка в current_listevents_creator_id

Тут опечатка в `current_listevents_creator_id`
ivan.dev marked this conversation as resolved
ivan.dev added 1 commit 2025-07-04 11:30:12 +05:00
ivan.dev changed title from WIP: feat: CRUD ListEvent to feat: CRUD ListEvent 2025-07-04 18:04:03 +05:00
ivan.dev added 1 commit 2025-07-04 18:05:46 +05:00
ivan.dev merged commit d03600b23d into master 2025-07-04 18:06:03 +05:00
Sign in to join this conversation.
No reviewers
No Label
No Milestone
No project
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Vorkout/connect#14
No description provided.