Таска 1/2: явный каст и 404 ошибка #1

Open
skvachuk wants to merge 3 commits from feature/1 into develop
Collaborator

Чутка затупил и закоммитил 404 в обе ветки, так что пушу рефактор только в этой ветке

Чутка затупил и закоммитил 404 в обе ветки, так что пушу рефактор только в этой ветке
skvachuk changed target branch from main to develop 2026-05-28 13:12:52 +00:00
skvachuk added 1 commit 2026-05-28 13:12:52 +00:00
damanukyan requested changes 2026-05-28 17:36:23 +00:00
@@ -53,3 +56,3 @@
user = await command(user_id=auth.user_id)
logger.info(f'Get user: {user.id}')
return me_user_public(user)
return MeUserPublicResponse.from_user(user)
Owner

Лучше сделать явный каст сущности в респонз модель

не MeUserPublicResponse.from_user(user)

а типо что-то:
MeUserPublicResponse(
id=user.id,
...
)

Лучше сделать явный каст сущности в респонз модель не MeUserPublicResponse.from_user(user) а типо что-то: MeUserPublicResponse( id=user.id, ... )
@@ -198,3 +197,3 @@
) -> MeUserPublicResponse:
user = await command(user_id=auth.user_id)
return me_user_public(user)
return MeUserPublicResponse.from_user(user)
Owner

Лучше сделать явный каст сущности в респонз модель

не MeUserPublicResponse.from_user(user)

а типо что-то:
MeUserPublicResponse(
id=user.id,
...
)

Лучше сделать явный каст сущности в респонз модель не MeUserPublicResponse.from_user(user) а типо что-то: MeUserPublicResponse( id=user.id, ... )
@@ -1 +0,0 @@
from src.presentation.serializers.me_user import me_user_payload, me_user_public
Owner

Если папка осталась, то ее тоже удалить

Если папка осталась, то ее тоже удалить
skvachuk added 1 commit 2026-05-28 17:52:09 +00:00
skvachuk added 1 commit 2026-05-29 11:34:25 +00:00
damanukyan requested changes 2026-05-31 07:20:41 +00:00
@@ -179,1 +193,4 @@
updated_at=user.updated_at,
kyc_verified_at=user.kyc_verified_at
)
return SetAvatarPublicResponse(**pub.model_dump(), webp_size_bytes=webp_size)
Owner

**pub.model_dump()

Точно также нужно заменить на явное создание, а не распаковку

**pub.model_dump() Точно также нужно заменить на явное создание, а не распаковку
This pull request can be merged automatically.
You are not authorized to merge this pull request.
View command line instructions

Checkout

From your project repository, check out a new branch and test the changes.
git fetch -u origin feature/1:feature/1
git checkout feature/1
Sign in to join this conversation.
No Reviewers
No Label
2 Participants
Notifications
Due Date
No due date set.
Dependencies

No dependencies set.

Reference: damanukyan/users#1