Sprint 2 #5
Sprint 2 #5
Conversation
✅ Deploy Preview for legendary-cat-a62c0e ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
There was a problem hiding this comment.
Здравствуйте. (Нужно развернуть общий комментарий ↓)
Посмотрите на гитхабе все комментарии к коду (нужно прокрутить вниз страницу там)
Работа проделана огромная
Что сделано отлично:
Readmeхорошо оформлен- Проект задеплоен и отлично работает
- Отлично, что не забыли про
.gitignore - Отлично, что ограничиваете версию
node - Отлично, что нет
EOFошибок в гите - Хорошая структура папок и файлов
- Добавлен
ESLintиStylelint - Отлично, что цвета вынесены в
css-константы - Отлично, что не забываете про тег
mainдля главной части страницы - Отлично, что не забываете удалять обработчики
- Отличная реализация валидации
Что можно улучшить:
- лучше всего передавать тип пропсов в виде дженерика в
Block, чтобы можно было конкретный тип пропсов всегда иметь под каждый отдельный экземпляр классаBlock. Будет очень универсально
Поздравляю! Ваша работа принята.
Критических замечаний нет
Вы отлично потрудились.
Удачного дальнейшего обучения.
| }) | ||
| } | ||
|
|
||
| private _removeEvents() { |
There was a problem hiding this comment.
Отлично
Отлично, что не забываете удалять обработчики
| export type Props = Record<string, unknown>; | ||
| export type Children = Record<string, Element | Block>; | ||
|
|
||
| class Block { |
There was a problem hiding this comment.
Можно лучше
лучше всего передавать тип пропсов в виде дженерика в Block, чтобы можно было конкретный тип пропсов всегда иметь под каждый отдельный экземпляр класса Block. Будет очень универсально
abstract class Block<Props extends Record<string, any> = unknown> {| timeout?: number, | ||
| } | ||
|
|
||
| export type HTTP = (url: string, options?: Options) => Promise<any>; |
There was a problem hiding this comment.
Отлично
Отлично, что не дублируете типизацию аргументов
No description provided.