EventUser table by Georgon · Pull Request #124 · profcomff/timetable-api

Georgon · 2026-04-20T13:43:46Z

Добавил новый класс (таблицу) EventUser для задания событий у пользователя. Каждое событие может обладать следующими статусами:
NO_STATUS (автоматические статус для всех событий),
GOING - статус события, на которое пользователь идет,
NOT_GOING - статус события, на которое пользователь не идет,
ATTENDED - статус события, которое пользователь отметил как GOING и время конца события уже прошло.

Для всех типов статусов также организован класс EventUserStatus.

Создан новый файл миграции. Все проверил, табличка создается.

Closes #121

github-actions · 2026-04-20T13:44:17Z

💩 Code linting failed, use black and isort to fix it.

github-actions · 2026-04-20T13:44:49Z

Coverage Report

File	Stmts	Miss	Cover	Missing
calendar_backend
__main__.py	6	6	0%	1–16
exceptions.py	13	2	85%	8, 14
calendar_backend/methods
list_calendar.py	70	49	30%	28–51, 58–64, 71–76, 85–98, 102–113
utils.py	35	26	26%	14–19, 23–28, 34–38, 42–46, 52–56
calendar_backend/models
base.py	64	1	98%	87
calendar_backend/routes
base.py	74	7	91%	75, 80–81, 86–87, 98, 101
calendar_backend/routes/event
comment.py	47	6	87%	30, 41–43, 52, 64
comment_review.py	27	4	85%	23–29, 41
event.py	112	8	93%	38, 45–48, 53, 113, 117
user_event.py	16	8	50%	21–42
calendar_backend/routes/group
group.py	38	3	92%	26, 42, 58
calendar_backend/routes/lecturer
comment.py	48	8	83%	35, 49, 57–61, 70
comment_review.py	25	4	84%	21–29, 41
lecturer.py	60	8	87%	27, 41, 44, 54, 78–81
photo.py	43	5	88%	43, 48, 65, 76, 79
photo_review.py	51	20	61%	39–58, 75–77
calendar_backend/routes/models
base.py	60	9	85%	36–40, 43, 56, 67, 80
event.py	62	3	95%	15, 42, 60
group.py	39	16	59%	13–19, 22, 32–38, 41
lecturer.py	46	2	96%	21, 31
room.py	21	1	95%	13
calendar_backend/routes/room
room.py	39	3	92%	26, 44, 58
TOTAL	1202	199	83%

Summary

Tests	Skipped	Failures	Errors	Time
34	0 💤	0 ❌	0 🔥	4.178s ⏱️

github-actions · 2026-04-20T13:47:56Z

💩 Code linting failed, use black and isort to fix it.

petrCher · 2026-04-20T14:13:35Z

https://github.com/profcomff/rental-api/blob/main/rental_backend/models/__init__.py#L5
посмотри сюда. надо будет добавить класс новый в свой инит

…ble-api into EventUser_table

github-actions · 2026-04-20T14:58:46Z

💩 Code linting failed, use black and isort to fix it.

github-actions · 2026-04-20T15:01:45Z

💩 Code linting failed, use black and isort to fix it.

…ble-api into EventUser_table

github-actions · 2026-04-23T15:14:02Z

💩 Code linting failed, use black and isort to fix it.

Georgon · 2026-04-23T15:26:02Z

Добавил новый файл routes/event/user_event.py, где реализована ручка post event/{event_id}/visit/.
Для нее добавлены две Pydentic модели VisitRequest и VisitResponse в отдельном файле visit.py.
В routes/base.py подключил роутер user_event_router.

Логика ручки:

Получаем id пользователя + проверка на авторизацию
Проверка, что событие существует и не удалено
Поиск/создание записи о посещении (с изменением статуса)

Также в прошлый раз забыл добавить в calendar_backend/models/init.py класс EventUserStatus.

github-actions · 2026-04-23T15:29:50Z

💩 Code linting failed, use black and isort to fix it.

Georgon · 2026-04-23T15:33:13Z

Closes #119

petrCher · 2026-04-23T17:46:47Z

+
+@router.post("/{event_id}/visit", response_model=VisitResponse)
+async def set_event_visit_status(
+    event_id: int, visit: VisitRequest, auth: dict = Depends(UnionAuth(scopes=[]))


scopes=[]
удалить

petrCher · 2026-04-23T17:47:04Z

+    """
+    Отметить посещение мероприятия для текущего пользователя.
+    """
+    user_id = auth.get('id')


удалить авторизацию

petrCher · 2026-04-23T17:48:37Z

+    event_id: int
+    user_id: int
+    status: EventUserStatus
+    updated_at: str  # ISO-формат


поменять на datetime
аналогично ренталу

petrCher · 2026-04-23T17:51:19Z

+
+    if existing:
+        existing.status = visit.status
+        db.session.flush()


работа с бд в отдельном классе BaseDbModel

petrCher · 2026-04-23T19:16:34Z

+    try:
+        _ = Event.get(event_id, with_deleted=False, session=db.session)
+    except ObjectNotFound:
+        raise HTTPException(


у нас есть прям отдельный класс для ошибки ObjectNotFound, импортирую его из exceptions и используй его

petrCher · 2026-04-23T19:17:11Z

+        )
+
+    try:
+        _ = Event.get(event_id, with_deleted=False, session=db.session)


необязательно делать присвоение пустой переменной, можно просто сделать ивент.гет

и кстати посмотри, если не ошибаюсь то сам нет запрос от класса при отсутствии соответствующей переданной переменной сам вызовет 404 ошибку, так что запрос на ошибку ниже вероятно лишний

petrCher · 2026-04-23T19:18:54Z

+        )
+
+    existing = (
+        db.session.query(EventUser)


сделать EventUser.query и тд

petrCher · 2026-04-23T19:23:58Z

+
+    db.session.commit()
+
+    return VisitResponse(


лучше делать VisitResponce.model_validate() реализацию можно посмотреть буквально почти во всех ручках рентала или рейтинга

petrCher · 2026-04-23T19:31:22Z

+
+@router.post("/{event_id}/visit", response_model=VisitResponse)
+async def set_event_visit_status(
+    event_id: int, visit: VisitRequest, auth: dict = Depends(UnionAuth(scopes=[]))


https://github.com/profcomff/rating-api/blob/main/rating_api/routes/comment.py#L205-L208
вот как список реализовать

github-actions · 2026-04-25T14:57:49Z

💩 Code linting failed, use black and isort to fix it.

petrCher · 2026-04-25T16:08:19Z

+async def set_event_visit_status(
+    event_id: int,
+    auth: dict = Depends(UnionAuth()),
+    visit: str = Query(enum=["no_status", "going", "not_going", "attended"], default="no_status"),


чисто из логики, attended надо убрать, так как пользователь не должен иметь возможность поставить инфу что посещена пара, нелогично

petrCher · 2026-04-25T16:18:06Z

+    """
+    user_id = auth.get('id')
+
+    Event.get(event_id, with_deleted=False, session=db.session)


with_deleted=False необязательно прописывать, по умолчанию false

petrCher · 2026-04-25T16:19:04Z

+            status=visit,
+        )
+
+    db.session.commit()


это лишнее, убрать

petrCher · 2026-04-25T16:29:04Z

+    status: EventUserStatus
+
+
+class VisitResponse(BaseModel):


наследованные брать от класса Base из соседнего файла Base, там уже прописано model_config = {"from_attributes": True}

petrCher · 2026-04-25T16:29:13Z

+from calendar_backend.models import EventUserStatus
+
+
+class VisitRequest(BaseModel):


поменять на Base

petrCher · 2026-04-25T16:29:21Z

+    status: EventUserStatus
+    updated_at: datetime.datetime
+
+    model_config = {"from_attributes": True}


убрать

EventUser table

35ad55b

Georgon requested a review from petrCher April 20, 2026 13:43

Georgon self-assigned this Apr 20, 2026

Georgon added this to Viribus Team Apr 20, 2026

EventUser table with right format

397c116

petrCher added 2 commits April 20, 2026 16:58

black format

c272363

bug fix

5254935

petrCher requested changes Apr 20, 2026

View reviewed changes

Comment thread calendar_backend/models/db.py Outdated

Comment thread calendar_backend/models/db.py Outdated

petrCher removed this from Viribus Team Apr 20, 2026

Georgon added 3 commits April 20, 2026 17:33

EventUser init

f21e312

Merge branch 'EventUser_table' of https://github.com/profcomff/timeta…

9513e1f

…ble-api into EventUser_table

EventUser fix

7d4d213

EventUser fixed with format

2254c3c

petrCher and others added 3 commits April 20, 2026 18:42

bug fix

ad6c63b

post event/{event_id}/visit/

aa6f76a

Merge branch 'EventUser_table' of https://github.com/profcomff/timeta…

ecddbdd

…ble-api into EventUser_table

fixed user_event

6094ad4

user_event formatted

c0d45e9

petrCher requested changes Apr 23, 2026

View reviewed changes

petrCher linked an issue Apr 23, 2026 that may be closed by this pull request

сделать post event/{event_id}/visit/ #119

Open

petrCher requested changes Apr 23, 2026

View reviewed changes

user_event fixed

08859e4

user_event formatted

eb7ba9e

petrCher requested changes Apr 25, 2026

View reviewed changes

		from calendar_backend.models import EventUserStatus


		class VisitRequest(BaseModel):

Conversation

Georgon commented Apr 20, 2026

Uh oh!

github-actions Bot commented Apr 20, 2026

Uh oh!

github-actions Bot commented Apr 20, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Summary

Uh oh!

github-actions Bot commented Apr 20, 2026

Uh oh!

Uh oh!

Uh oh!

petrCher commented Apr 20, 2026

Uh oh!

github-actions Bot commented Apr 20, 2026

Uh oh!

github-actions Bot commented Apr 20, 2026

Uh oh!

github-actions Bot commented Apr 23, 2026

Uh oh!

Georgon commented Apr 23, 2026

Uh oh!

github-actions Bot commented Apr 23, 2026

Uh oh!

Georgon commented Apr 23, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

github-actions Bot commented Apr 25, 2026

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Choose a reason for hiding this comment

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions Bot commented Apr 20, 2026 •

edited

Loading