Skip to content
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

Create Swagger descriptions for volunteer api #185

Open
wants to merge 6 commits into
base: development
Choose a base branch
from

Conversation

zoomkolol
Copy link
Contributor

No description provided.

@zoomkolol zoomkolol linked an issue Sep 15, 2024 that may be closed by this pull request
4 tasks
@zoomkolol zoomkolol requested a review from kspshnik September 15, 2024 20:49
Copy link
Collaborator

@kspshnik kspshnik left a comment

Choose a reason for hiding this comment

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

Очень хорошо, но 400 надо бы детальнее обругать :))

@ApiCreatedResponse({ type: Promise<Task[]> })
@ApiUnauthorizedResponse({ description: 'Требуется авторизация' })
@ApiForbiddenResponse({ description: 'Требуется другой статус или роль' })
@ApiBadRequestResponse({ description: 'Произошла ошибка' })
Copy link
Collaborator

Choose a reason for hiding this comment

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

А точно просто "ошибка"? Или есть конкретный косяк в запросе?

@ApiCreatedResponse({ type: Promise<Task> })
@ApiUnauthorizedResponse({ description: 'Требуется авторизация' })
@ApiForbiddenResponse({ description: 'Требуется другой статус или роль' })
@ApiBadRequestResponse({ description: 'Произошла ошибка' })
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично предыдущему.

@zoomkolol zoomkolol requested a review from kspshnik September 19, 2024 19:40
@ApiOperation({ summary: 'Получение новых задач по статусу' })
@ApiQuery({ type: GetTasksSearchDto })
@ApiCreatedResponse({ type: Promise<Task[]> })
@ApiUnauthorizedResponse({ description: 'Требуется авторизация' })
Copy link
Collaborator

Choose a reason for hiding this comment

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

Нужно не только сделать описание, но и показать пример ответа, т.е. при ошибке у нас выбросится объект вида...

@zoomkolol zoomkolol requested a review from INextYP October 15, 2024 19:41
@zoomkolol
Copy link
Contributor Author

@kspshnik @INextYP обновил описания

Copy link
Collaborator

@kspshnik kspshnik left a comment

Choose a reason for hiding this comment

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

Сейчас слияние заблокировано: надо подтянуть последние изменения из development и решить все конфликты.

@zoomkolol zoomkolol requested a review from kspshnik October 16, 2024 18:45
@kspshnik kspshnik added backend Задача для бэкенда PR Pull Request labels Oct 19, 2024
Copy link
Collaborator

@kspshnik kspshnik left a comment

Choose a reason for hiding this comment

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

Рома, надо нормально описать ошибку 400.
Например, на ручке PUT /recipient/tasks/:id/accept ошибка 400 возникнет, если :id не пройдёт валидацию, на ручке GET /volunteer/tasks/accepted' - если не отвалидируется search query (описанное GetTasksSearchDto`).
Именно это и надо указать в описании ошибки в Swagger.

}
})
@ApiBadRequestResponse({
description: 'Ошибка запроса. В нём есть пользователь?',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Нет, надо внятно объяснить в чём проблема. В каких случаях летит 400? Проанализируй код контроллера и dto, и распиши, какие возможны ошибки.

}
})
@ApiBadRequestResponse({
description: 'Ошибка запроса. В нём есть пользователь?',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично.

}
}
})
@ApiBadRequestResponse({
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично

}
})
@ApiBadRequestResponse({
description: 'Ошибка запроса. В нём есть пользователь?',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично.

}
})
@ApiBadRequestResponse({
description: 'Ошибка запроса. В нём есть пользователь?',
Copy link
Collaborator

Choose a reason for hiding this comment

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

Аналогично.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
backend Задача для бэкенда PR Pull Request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Volonteer Api. Описание ручек в Swagger
3 participants