Sprint 3

[KomAnw]

No description provided.

[netlify]

✅ Deploy Preview for sweet-messanger ready!

Name	Link
🔨 Latest commit	ef4b7d8
🔍 Latest deploy log	https://app.netlify.com/sites/sweet-messanger/deploys/63acd6e9bcaff300084b1d6d
😎 Deploy Preview	https://deploy-preview-3--sweet-messanger.netlify.app/
📱 Preview on mobile	Toggle QR Code... Use your smartphone camera to open QR code link.

---

To edit notification comments on pull requests, go to your Netlify site settings.

[gennady-bars]

Здравствуйте. (Нужно развернуть общий комментарий ↓)

Работа проделана огромная

• Readme хорошо оформлен
• Проект задеплоен и отлично работает
• Отлично, что фиксируете версии зависимостей (почти все)
• Отлично, что нет EOF ошибок в гите
• Хорошая структура папок и файлов
• Отлично, что базовый урл вынесен в константу
• Хороший роутинг
• Хорошая логика сокетов

но есть некоторые недочеты:

• не могу запустить проект локально через npm start. Скрин https://disk.yandex.ru/i/PSsbAGwoE03M0A . И вот такие ошибки летят дальше https://disk.yandex.ru/i/efxw4R4qaqznIw
• Для любых запросов на сервер в самом конце цепочки запроса нужно осуществлять проверку на возможные ошибки с помощью catch. Нужно исправить это везде по коду в конце запросов.

Можно лучше

• лучше сделать подтверждение удаления чата, чтобы нечаянно не удалить мгновенно его. Это можно сделать с помощью prompt
• должна быть возможность изменять картинку для чата
• попробуйте избавиться от any

Исправьте, пожалуйста, недочеты и работа будет принята. Пожалуйста, проверьте работоспособность проекта и наличие возможных ошибок в консоли браузера (кнопка F12) перед отправкой на ревью.

Напоминаю, что работа может быть принята только после исправления всех критических замечаний Нужно исправить.

Удачного рефакторинга кода.

[KomAnw]

Привет! Спасибо за ревью! :)
По поводу ошибок сборки проекта о которых вы писали (https://disk.yandex.ru/i/efxw4R4qaqznIw):

1. ошибку с первого скрина я устранил
2. у меня локально все устанавливается и работает нормально, yarn run build/start работают корректно. На нетлифае также эти команты работают корректно... Я нашел только два треда с такой ошибкой, мне кажется что это проблема parcel... и то, что начинают устанавливаться не те зависимости(?)
   Прикрепляю ссылку на треды:
   Parcel 2: Expected constructor errors when building or watching parcel-bundler/parcel#3450
   Parcel doesn't work at all anymore (@parcel/workers: Expected constructor 2.0.0-nightly.793+91287030:BundleGraph to be registered with serializer to deserialize) parcel-bundler/parcel#6675

Думаю я бессилен в починке этих ошибок, может быть вам попробовать использовать не npm, а yarn, как я указал в документации к проекту?

P.S. это не единственная проблема parcel с которой я столкнулся, также на данный момент динамические импорты работают только в dev режиме, а в build режиме parcel не может соотнести хеши файлов из-за чего при подгрузке приложение крашится 👎

P.S.S. остальное из критичного поправил, сделал подтверждение на крестик. На остальное уже сил не хватает в этом году :D

[gennady-bars]

Поздравляю! Ваша работа принята.

Вы отлично потрудились.

Удачного дальнейшего обучения.

• в 4м спринте будет webpack. Там уже не должно быть проблем с yarn или билдом