Difficulty of challenges #12
Conversation
…лись сразу после завершения
| @@ -0,0 +1,8 @@ | |||
| namespace Models.Challenges; | |||
|
|
|||
| public enum ChallengeDifficulty | |||
There was a problem hiding this comment.
Если поле с типом енума пишется в базу как int, то лучше явно указывать какому значение какое число соответствует. Так проще поддерживать обратную совместимость если потом будут добавляться новые значения.
There was a problem hiding this comment.
Ну или явно делать поле в DBO интовым и явно писать конвертацию)
| aria-errormessage={error?.message} | ||
| > | ||
| <Stack direction="row"> | ||
| <Radio value="easy">Лёгкая</Radio> |
There was a problem hiding this comment.
Как и обсуждали, лучше для поддержания обратной совместимости сделать чтобы у требований могла отсутствовать сложность. Можно или сделать поле nullable, или явно добавить значение unknown, которое маппится в 0.
There was a problem hiding this comment.
Думаю можно оставить здесь только 3 кнопки, чтобы у новых требований всегда указывалась сложность, но лучше не делать его обязательным чтобы после релиза можно было заполнять сложность постепенно.
Добавил поле difficulty в модель требования.
Добавил выбор сложности с помощью радио-кнопок в форме создания/редактирования требования.