
Artyom
16.06.2017
06:23:27
Всем привет. Кто хочет провести код ревью моего junior кода?)))
Приложуха в папке ReduxApp.
https://github.com/artalar/search-jobs-react-demo

? ethorz
16.06.2017
06:23:37
вангую, что должна быть какая-то глобальная переменная

K
16.06.2017
06:23:48
вебпаком
Ну наверн надо написать простой вебпак плагин, который регуляркой будет искать нужное место и вставлять
Ну или это велосипед, но готовых решений не знаю

Google

Stepan
16.06.2017
06:24:20
Я ручками константу везде ставлю

Vladislav
16.06.2017
06:24:32
это nuclide-server
ради него он изначально и создавался - под фейсбуковский процесс разработки

? ethorz
16.06.2017
06:25:13
автоматизировать, так сказать

Stepan
16.06.2017
06:26:36
Ну, у меня 3 разных апишки
Тут автоматизировать не выйдет
Webpack
Обсуждения webpack 1/2, лоадеров, плагинов и сборок
https://t.me/webpack_ru

? ethorz
16.06.2017
06:28:34
вставил в файл process.env.NODE_ENV
показывает development / production
но нет
при сборке, можно как-то передавать параметр
который потом получить в js файле?
ладно, пойду в тот чат

Google

Alexey
16.06.2017
06:35:24

? ethorz
16.06.2017
06:37:01
ахах)
https://github.com/artalar/search-jobs-react-demo/blob/beta/ReduxApp/reducers/index.js
зачем столько констант? Если они делают одно и то же

lamo2k
16.06.2017
06:38:21
Я все еще не вижу проблемы
Как ты его возьмёшь если запрос на сервер происходит раньше чем выполнение JS который сможет достать данные из бокал стораджа

Ywein
16.06.2017
06:39:22

lamo2k
16.06.2017
06:41:41

(;¬_¬)
16.06.2017
06:42:16
дык табы поди

Ywein
16.06.2017
06:42:29

lamo2k
16.06.2017
06:43:54
Если пользователь авторизовался и его токен не протух то с бэка должна вернуться страница с его данными.

Ywein
16.06.2017
06:48:41

lamo2k
16.06.2017
06:52:24

Artyom
16.06.2017
06:52:30
В VSC красиво :)

lamo2k
16.06.2017
06:56:15

Ywein
16.06.2017
06:57:00

lamo2k
16.06.2017
06:57:36
А почему деструктором по одной константе забирает е?

Artyom
16.06.2017
06:58:47
Пробовал классически писать, from в этом случае не нравится, как выглядит. А так всегда четкая линия, что и откуда

Google

Сергей
16.06.2017
06:59:02

lamo2k
16.06.2017
07:02:32
Ещё есть вот такая штука https://www.npmjs.com/package/redux-actions-helpers может быть мега полезной

Artyom
16.06.2017
07:06:44

Сергей
16.06.2017
07:14:17
Ещё redux-act и redux-actions

Oleg ?
16.06.2017
07:30:45
Redux-actions мне больше нравится
Скорость разработки с ним выросла в два раза

Artur
16.06.2017
07:33:19

Oleg ?
16.06.2017
07:33:50
Скорость появления багов в 10?)

Aleksey
16.06.2017
07:34:05

Artur
16.06.2017
07:34:40

Oleg ?
16.06.2017
07:35:29
Что если у меня в команде есть любители мутировать по страшному объекты?

Artur
16.06.2017
07:35:33
Ну и код сразу оптимизированный, shpuldComponentUpdate писать не приходится от слова вообще

Oleg ?
16.06.2017
07:35:44
Как он отследит изменения?

Сергей
16.06.2017
07:35:44
Ясно понятно

Artur
16.06.2017
07:36:01

Oleg ?
16.06.2017
07:36:03

Artur
16.06.2017
07:36:25

Google

Oleg ?
16.06.2017
07:36:50
Не понимаю как это связано

Aleksey
16.06.2017
07:36:57
Как ты будешь контроллировать выстреливающие из всех возможных мест фейерверков?

Artur
16.06.2017
07:37:03
Я тоже поначалу скептически отнесся, пока не попробовал

anoru
16.06.2017
07:37:14
о, срачик

Artur
16.06.2017
07:37:16

Oleg ?
16.06.2017
07:37:29
Ещё раз задам вопрос: кто - то изменил свойство объекта, как это отследить?

Artur
16.06.2017
07:37:54

anoru
16.06.2017
07:38:02
омг

Admin
ERROR: S client not available

Oleg ?
16.06.2017
07:38:15
А делает это он за счёт обсервабл

Artur
16.06.2017
07:38:22
Именно так

Konstantin
16.06.2017
07:38:29

Oleg ?
16.06.2017
07:38:42
А теперь, что делать с nested object, который лень покрывать всеми обсервами?

Aleksey
16.06.2017
07:38:46
О чем ты вообще?
Я о безумстве обсервов. Я это проходил с нокаутом, это прикольно, но до момента когда не ловишь какую нибудь дикую хуйню.

Artur
16.06.2017
07:39:03

Konstantin
16.06.2017
07:39:20

Artur
16.06.2017
07:39:35

Default
16.06.2017
07:39:52

Oleg ?
16.06.2017
07:39:58

Artur
16.06.2017
07:40:09

Google

Oleg ?
16.06.2017
07:40:13
А ещё они их будут мутировать

Artur
16.06.2017
07:40:29

Konstantin
16.06.2017
07:40:35

Oleg ?
16.06.2017
07:41:04

Ruslan
16.06.2017
07:41:10

Oleg ?
16.06.2017
07:41:40
И пока они есть, у нас всё нормально и багов, которые сложно отследить не возникает

Artur
16.06.2017
07:42:37

Konstantin
16.06.2017
07:43:03

Oleg ?
16.06.2017
07:43:13
У нас с этим не было проблем, мы используем место мемоизированые селекторы

Artur
16.06.2017
07:43:18
https://m.habrahabr.ru/post/304340/
Читать до бонусного совета

Konstantin
16.06.2017
07:43:49

Oleg ?
16.06.2017
07:44:17

Artur
16.06.2017
07:44:19

Konstantin
16.06.2017
07:44:52

Artur
16.06.2017
07:44:54
Паттерны ради паттернов и так далее
И не нужны ни селекторы, ни мемоизированнык селекторы, ничего. Оно простото раьотает, сразу и быстро
Опечатки, сорян

Oleg ?
16.06.2017
07:46:28