Skip to content
This repository has been archived by the owner on Feb 2, 2022. It is now read-only.

Добавить валидацию #17

Closed
grigori-gru opened this issue Sep 12, 2019 · 3 comments
Closed

Добавить валидацию #17

grigori-gru opened this issue Sep 12, 2019 · 3 comments
Labels
backend Improvements or additions to documentation good first issue Good for newcomers
Milestone
MVP

Comments

@grigori-gru
Copy link
Contributor

Нужно добавить валидацию dto и конфига, это легко сделать как первый коммит для новичка. Через class-validator, конечно.
@jougene
Copy link
Contributor

jougene commented Sep 12, 2019

Добавить валидацию в конфиг можно, единственное я бы не стал называть это штуку ДТО, потому что немного странно выглядит.

@jougene jougene added backend Improvements or additions to documentation good first issue Good for newcomers labels Sep 12, 2019
@jougene jougene added this to the MVP milestone Sep 12, 2019
@grigori-gru
Copy link
Contributor Author

Я имею в виду дто на создание и обновление, они же сейчас есть уже в проекте. У конфига дто нет, конечно

@jougene
Copy link
Contributor

jougene commented Sep 14, 2019

Для создания заявки валидация добавлена, остальные валидации лучше делать в рамках конкретных фич.

@jougene jougene closed this as completed Sep 14, 2019
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Assignees
No one assigned
Labels
backend Improvements or additions to documentation good first issue Good for newcomers
Projects
None yet
Milestone
MVP
Development

No branches or pull requests
2 participants
@jougene @grigori-gru