50: add an error message for avatars#73
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
unknownproperty
left a comment
unknownproperty
left a comment
There was a problem hiding this comment.
Я предлагаю следующим образом поменять компонент:
– Полностью убрать внутренний стейт для ошибок и их отображение, оно как будто лишнее и ломает вёрстку, при этом в текущих правках, мы получаем раздвоение логики, когда есть и внутренние ошибки и внешние
– Все ошибки обрабатывать через один колбэк onError, но сделать возможность и отключить вовсе обработку ошибок через второй проп
– handleFileChange убрать, ибо у тебя две функции с почти одинаковыми названиями
– Вместо этого сделать функцию обработчик ошибок, где как раз будут проводится проверки и вызваться onError, если есть ошибка. При этом, можно вызывать onError и делать return, чтобы из компонента не возвращался файл
– Функцию выше использовать перед handleFilesChange
– Также стоит разобраться со специфическими проверками по типу файла или изображения, так как не всегда такие проверки нужны и лучше их задавать из вне через пропс. Например, если у тебя используется FileUploder для загрузки pdf, то мы должны валидировать на pdf
| disabled, | ||
| isWarning, | ||
| onChange, | ||
| isError = () => {}, |
There was a problem hiding this comment.
функции называем иначе
|
|
||
| const pluralFiles = ['файла', 'файлов', 'файлов']; | ||
|
|
||
| const readFile = (file: File) => |
There was a problem hiding this comment.
Это лишнее, далеко не все файлы нам нужно читать и превращать в base64
unknownproperty
left a comment
unknownproperty
left a comment
There was a problem hiding this comment.
Нужно перед мержем сделать npm i в репозитории
2 participants
No description provided.