-
Notifications
You must be signed in to change notification settings - Fork 1
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
docs(api-admin): add add-user method description | openapi docs #37
Conversation
ACC-61 Добавление пользователя в приложение
Отправляем application id, user id, возвращает успех или нет (описать варианты ошибок). Пользователь не может зарегистрироваться в приложении дважды, но может быть зарегистрирован не входив ни разу |
api-admin/openapi.yaml
Outdated
enum: | ||
- access_denied | ||
- invalid_request | ||
- temporarily_unavailable |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Для этой ошибки есть отдельный статус 500
- user_not_found | ||
- unauthenticated_user | ||
- unauthorized_client | ||
redirectUri: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Лишнее поле, кажется
api-admin/openapi.yaml
Outdated
description: User should be redirected to if passed redirectUri and clientId is correct | ||
type: string | ||
format: url | ||
state: |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Это тоже лишнее
api-admin/openapi.yaml
Outdated
application/json: | ||
schema: | ||
required: | ||
- error |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Обязательное поле error, но поля такого нет, есть code
api-admin/openapi.yaml
Outdated
- application_not_found | ||
- user_not_found | ||
- unauthenticated_user | ||
- unauthorized_client |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Думаю, такой ошибки не будет
docs(api-admin): fix path docs(api-admin): fix review comments docs(api-admin): fix review comments docs(api-admin): fix error description Create linear-linker.yml docs(api-admin): add add-user method description | openapi docs (#37) fix(api-admin): naming, remove example method merged manually #39 Closes #39 fix docs(api-admin): add delete user method description | openapi docs (#36) * docs(api-admin): add delete user method description | openapi docs * docs(api-admin): fix review comments * docs(api-admin): fix review * docs(api-admin): typo fixes * docs(api-admin): fix error description Apply suggestions from code review
Описание контракта для ручки Добавления пользователя в приложение. Задача -> https://linear.app/atomix/issue/ACC-61/dobavlenie-polzovatelya-v-prilozhenie