Яковлев Илья, М3339 #15
Conversation
|
This pull request is automatically deployed with Now. Latest deployment for this branch: https://task-5-git-fork-okayvlev-master.itmo-yandex.now.sh |
src/app/main/mailbox/data.ts
Outdated
|
|
||
| export const PRELOADED_MAILS = [ | ||
| new Mail( | ||
| 'images/owl-face.jpg', |
There was a problem hiding this comment.
При таком подводе может быть очень неудобно поддерживать код. Лучше передавать в функцию объект с соответствующими именованными полями
types/react.d.ts
Outdated
| @@ -0,0 +1,50 @@ | |||
| export namespace Children { | |||
There was a problem hiding this comment.
Чтобы не делать так, и соответственно не ломать всю типизацию, нужно установить @types/react
There was a problem hiding this comment.
Я так понял ты просто переопределился чтобы не было проблем с типизацией в реакте, но это по сути отключение типизации. Так не пойдёт
|
|
src/app/main/mailbox/mailbox.tsx
Outdated
| mails: Array<Mail>; | ||
| } | ||
|
|
||
| export class MailBox extends Component<any, MailBoxState> { |
There was a problem hiding this comment.
Какой смысл везде использовать any? Приложения на TypeScript не используют или почти не используют any т.к. это отключает всю проверку и теряет смысл
rmbaad
left a comment
rmbaad
left a comment
There was a problem hiding this comment.
Вижу только 1 задание, инпут для поиска вообще не является инпутом
src/app/main/mailbox/mail/mail.tsx
Outdated
| onAnimationEnd: () => void; | ||
| } | ||
|
|
||
| export class Mail extends Component<MailProps, any> { |
There was a problem hiding this comment.
Если нет стейта, то второй параметр для Component просто можно не указывать
2 participants
No description provided.