поверхностный скан - вроде все ок, с другой стороны, на всех инпутах почему-то фокус/блюр слушается, хотя используется только в дейтпикере, у всех инпутах одинаковые айдишники id и лейблы for=id у всех проставлен. Ошибку показывать во втором лейбле тоже не сильно семантично, кмк. Структура папок, я так понял, задается накстом или каким-то генератором, первая мысль "как-то все сильно наружу торчит". На кодревью бы предлагал нейминг для некоторых вещей поменять. В сторе API взаимодействие и эррор хендлинг чуток специфичен. store/dom мертвый код, что ли?

Ой, вибачте, затупив, не побачив що по дефолту приватні репозиторії створюються, поправив
І створив github, може комусь там зручніше
https://bitbucket.org/shtyegor/test
https://github.com/YegorShtonda/test

Я никогда с этим стеком не работаал, но если придираться:

1) Почему-то микс из await и промисов. Выглядит сранно. (Вопрос к тебе, как правильно захендлить ошибку при использовании await) https://bitbucket.org/shtyegor/test/src/50905027b808b4db41f4f1c62f501f997efc53d5/store/products.js#lines-28:40

2) Не очень хороший алгоритм поиска (вопрос к тебе, объясни почему он не оптимальный) . Думаю, что стоит использовать .find. https://bitbucket.org/shtyegor/test/src/50905027b808b4db41f4f1c62f501f997efc53d5/store/products.js#lines-50:58

3) Вероятно стоит назвать isPopupOpen https://bitbucket.org/shtyegor/test/src/50905027b808b4db41f4f1c62f501f997efc53d5/store/popups.js#lines-21

Дякую, мої пройоби, більше так не буду
1) Напевно викидати new Error(хоча це питання мене трохи в кут загнало, тре більше про async/await і промізи почитати)
2) Так find, мені стидно за те

Эта штука с тайпскриптом вообще дружит? Очень много строковой магии, как в первом ангуляре 🙂

Эта штука с тайпскриптом вообще дружит? Очень много строковой магии, как в первом ангуляре 🙂

Как это работает, какая-то магия вьюшная? https://github.com/YegorShtonda/test/search?q=fetchProducts&unscoped_q=fetchProducts

fetchProducts нигде не объявлена в сторах

ніт

Как это работает, какая-то магия вьюшная? https://github.com/YegorShtonda/test/search?q=fetchProducts&unscoped_q=fetchProducts

fetchProducts нигде не объявлена в сторах

Ой, вибачте, затупив, не побачив що по дефолту приватні репозиторії створюються, поправив
І створив github, може комусь там зручніше
https://bitbucket.org/shtyegor/test
https://github.com/YegorShtonda/test

А что значит ни где в строках не обявляенна?

Сторах (store), не строках 🙃

поверхностный скан - вроде все ок, с другой стороны, на всех инпутах почему-то фокус/блюр слушается, хотя используется только в дейтпикере, у всех инпутах одинаковые айдишники id и лейблы for=id у всех проставлен. Ошибку показывать во втором лейбле тоже не сильно семантично, кмк. Структура папок, я так понял, задается накстом или каким-то генератором, первая мысль "как-то все сильно наружу торчит". На кодревью бы предлагал нейминг для некоторых вещей поменять. В сторе API взаимодействие и эррор хендлинг чуток специфичен. store/dom мертвый код, что ли?

АК обявляенна же)

Я не знаю как оно должно быть, я с вью никогда не работал, но он вроде ссылается на "products/fetchProducts", а в сторе есть метод fetch, но нет метода fetchProducts