Conversation
korzhevmd24
force-pushed
the
misha_renaming_swagger
branch
from
c46ad6f to
19e49cc
Compare
|
💩 Code linting failed, use |
Summary
|
|
💩 Code linting failed, use |
|
💩 Code linting failed, use |
|
💩 Code linting failed, use |
petrCher
left a comment
petrCher
left a comment
There was a problem hiding this comment.
Наверху над моими комментами в коде написано comment on line . Обрати внимание если там не одна строка а несколько, чтобы не упустить ничего. Также я откомментил только один файл, в остальных почти все хорошо кроме аналогичных проблем с тем файлом, что я откомментил. Так что менять надо все файлы
| - список объектов `Event` | ||
|
|
||
| Returns a list of events. | ||
| Ошибки: |
There was a problem hiding this comment.
ахаххаха, забавно)
попробуй как минимум ввести -100
There was a problem hiding this comment.
ошибки отсутствуют лучше не писать никогда))
| Ошибки: | ||
| - отсутствуют |
There was a problem hiding this comment.
аналогично
плюс пробуй ввести туда например текст
| Условия: | ||
| - Пользователь должен быть аутентифицирован |
There was a problem hiding this comment.
убрать
ты же ниже про скупы написал, что доступно любому авторизованному пользователю
| Ошибки: | ||
| - `ObjectNotFound` — тип предмета с указанным `type_id` не найден |
There was a problem hiding this comment.
вот, в комментах выше что-то такое тоже надо делать
| Параметры: | ||
| - `item` — данные нового предмета |
There was a problem hiding this comment.
это неправда, там параметры type_id и is_available
| Условия: | ||
| - Пользователь должен быть аутентифицирован | ||
| - Пользователь должен иметь право на изменение предметов | ||
| - Предмет с указанным `id` должен существовать |
| - Пользователь должен быть аутентифицирован | ||
| - Пользователь должен иметь право на удаление предметов |
There was a problem hiding this comment.
только это убрать, про статусы оставить, там правильно
| - удаляется сам предмет | ||
| - удаляются связанные сессии аренды (если они не помечены как удалённые) | ||
| - удаляются связанные страйки и события | ||
| - действие логируется как `DELETE_ITEM` |
There was a problem hiding this comment.
логируется в event, из контекста это неочевидно
| Обновляет статус доступности предмета по его идентификатору. | ||
|
|
||
| Эндпоинт позволяет изменить только поле `is_available`. | ||
| После успешного обновления действие логируется как `UPDATE_ITEM`. |
There was a problem hiding this comment.
логируется в event, из контекста это неочевидно
| Условия: | ||
| - отсутствуют (доступно без авторизации) | ||
|
|
||
| Скоупы: | ||
| - отсутствуют |
There was a problem hiding this comment.
условия и скоупы удалить, зачем про них писать если не требуется?
|
Также сейчас падает один из тестов (linting). надо сделать форматирование black isort перед отправкой. Делается по команде make format(см мейкфайл) |
2 participants
Были добавлены описания для всех эндпоинтов в сваггере
Они включают в себя: