Skip to content

contract#1

Open
ks2999 wants to merge 5 commits into
mainfrom
ks2999-contract
Open

contract#1
ks2999 wants to merge 5 commits into
mainfrom
ks2999-contract

Conversation

@ks2999

@ks2999 ks2999 commented Jul 2, 2025

Copy link
Copy Markdown
Contributor

Контракт для UserService

Добавлен OpenAPI-контракт для UserService с поддержкой регистрации, аутентификации, профиля и ролей.

@ks2999 ks2999 added the documentation Improvements or additions to documentation label Jul 2, 2025
@ks2999 ks2999 self-assigned this Jul 2, 2025
@ks2999 ks2999 requested review from AxoRm and alexander-gileta July 2, 2025 13:14

@AxoRm AxoRm left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Добавьте аутентификацию как здесь, мы согласовали такой формат с командой, отвечающей за UI, думаю Вам тоже нужно это сделать.

@Shopin-Igor Shopin-Igor requested a review from MattoYuzuru July 3, 2025 10:08
@Shopin-Igor Shopin-Igor requested review from a team, AxoRm and EnotichEnotow July 3, 2025 10:09

@MattoYuzuru MattoYuzuru left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Норм, но не хватает эндпоинта для удаления юзера.
PATCH - изменение одного поля
PUT - изменение всех полей
А DELETE вообще нет

@vdenisov-pro vdenisov-pro requested review from a team and removed request for AxoRm, EnotichEnotow and alexander-gileta July 3, 2025 17:19
Правки по ревью:
- PATCH - изменение одного поля
- PUT - изменение всех полей
Добавил:
- DELETE
поправил название поля на старое, ошибочно поменял ранее
поправил название поля (password-> hashed_password)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants