
anoru
24.06.2016
11:23:51
нет обходных путей с текущим апи
Просто это очень важный кейс и непонятно как её юзают в своих проектах)
У себя сделал рендер в строку и сделал replace символа обратно в кавычку, адовый костыль

Alexander
24.06.2016
11:36:01
Повод поконтрибютить )

Roman
24.06.2016
11:36:35
Реально походу все стрингреплейсят

Google

Алексей
24.06.2016
11:38:00
>Hodor and Bran react to that massive

Alxander
24.06.2016
11:42:46
Там в выдаче куча реактовских сайтов с такими тайтлами.

anoru
24.06.2016
11:46:41
видать, helmet юзают))

Alxander
24.06.2016
11:47:06
Хелмет вообще костылём выглядит каким-то.

Ҫѐҏӗѫӑ
24.06.2016
11:48:24
https://jsfiddle.net/ggfk4dt0/
?

Roman
24.06.2016
11:52:25
Там другая фигня

Konstantin
24.06.2016
11:52:51
Есть вопрос? У меня есть данные. Формочка, все дела. И одна кнопка Save и надо проверить, если мол id в data есть то обновить данные если нет, то добавить новую запись. Во. Сейчас у меня два экшана onAdd.. onUpdate.. собственно вопрос? где логичней проверять какую операцию делать, в сторе и сделать один экшн или в компоненте и оставить два экшена?

Tony
24.06.2016
11:54:19
redux-thunk используешь?

Ҫѐҏӗѫӑ
24.06.2016
11:54:47
больные штоль такое юзать?
мета теги тоже в реакте добавляете?
есть компонент для консоль лога?

Google

Ҫѐҏӗѫӑ
24.06.2016
11:56:03
скрипт таги еще реактом тоже вставляйте, збс будет

Kich
24.06.2016
11:56:35
ахахаха

Roman
24.06.2016
11:56:37
Этот Хелмет во многих бойлерплейтах


anoru
24.06.2016
11:56:39
Хелмет и правда костылем выглядит. Непонятно зачем автор решил так делать. Если коротко, то хелмет может выдать данные в виде компонента или в виде сгенеренной хтмл строки. Однако у себя в механизмах он уже заменяет все ' на хтмл код этого.
Зачем? Нет логики. Видимо, автор хотел избежать конфликтов, когда кавычка открывается и тут же закрывается, часть данных выходит за предел. Но тут такого не произойдет, т.к. рендер ту стринг все равно все обернету тегов в двойную кавычку. То есть <meta content="" - будут двойные кавычки, а не одинарные. А значит эскейпить одинарные смысла нет.
поэтому если достать helmet.meta.title.toString() то он уже выдаст с замененой кавычкой на этот код.
Если же мы будем юзать helmet.meta.title.toComponent(), то проблема тоже останется. Т.к. он нам выдает уже именно реактовской компоеннт и опять же обработает кавычку.
Решить можно было бы, если бы автор дал возможность установить значение в тег через dangerSetInnerHtml, но такой возможности нет.
Решение первое:
форкаем либу и удаляем оттуда эскейп одинарной кавычки в код. Это не имеет смысла. Юзаем toString() и вставляем через <span dangerouslySetInnerHTML={></span> прямо в шапке
Решение второе:
достаем через helmet.meta.title.toString(). У нас теперь есть строкове представление всех мета тегов. И прямо их обрабатывает обратной заменой этого кода на нормальную кавычку


Ҫѐҏӗѫӑ
24.06.2016
11:56:51
головой надо думать же

anoru
24.06.2016
11:58:41
это просто странно, что helmet созданный для таких целей - сам же свои цели не выполняет. )

Kich
24.06.2016
11:59:37
странно, а чего у хелмета теги для head в пропсах пишутся, почему православным не jsx’ом?

Roman
24.06.2016
12:00:00
Чувак там в ишью сказал что это вообще проблема реакта и он ниче делать не будет :D

Ҫѐҏӗѫӑ
24.06.2016
12:00:12
чуваки, вы в курсе, что реакт - либа для ui?

anoru
24.06.2016
12:00:48

Kich
24.06.2016
12:01:00

anoru
24.06.2016
12:01:04
сек
https://github.com/nfl/react-helmet/blob/master/src/Helmet.js
в самое же начале)
он сам обрабатвыает одинарную кавычку

Roman
24.06.2016
12:01:29

anoru
24.06.2016
12:01:42
если это удалить будет уже лучше

Ҫѐҏӗѫӑ
24.06.2016
12:02:05
лол там и правда мета теги тоже предлагают реактом ставить

Google

Ҫѐҏӗѫӑ
24.06.2016
12:02:13
смишно

Kich
24.06.2016
12:02:25
ну дак

Ҫѐҏӗѫӑ
24.06.2016
12:02:28
лол и скрипт теги тоже
ой сука

anoru
24.06.2016
12:02:37
он просто для серверного рендеринга кул

Ҫѐҏӗѫӑ
24.06.2016
12:02:54
nfl engeneers не нужны

anoru
24.06.2016
12:02:54
позволяет установить мета теги в компоненте, апотом на ssr достать 1 строчкой
этим и подкупил)

Alexander
24.06.2016
12:03:52
Скоро не хватит места :c

anoru
24.06.2016
12:03:58
но спалил я его, конечно, у Эрика
https://github.com/erikras/react-redux-universal-hot-example/blob/master/package.json

Ҫѐҏӗѫӑ
24.06.2016
12:04:28

Ҫѐҏӗѫӑ
24.06.2016
12:04:32
лол

Kich
24.06.2016
12:04:39
вима нет на футболке. Отлично :)

Roman
24.06.2016
12:04:55
А в рск есть хелмет?

anoru
24.06.2016
12:05:12
вроде нет, Костя молодец)

Ҫѐҏӗѫӑ
24.06.2016
12:05:13
вим пусть будет

Kich
24.06.2016
12:05:32
Поддерживаю :)

Ҫѐҏӗѫӑ
24.06.2016
12:05:57
веб эссембли нужен(

Admin
ERROR: S client not available

Google

Nikita
24.06.2016
12:06:04
во-первых, хочу такую футболку)) во-вторых, а где JS-то?

Ҫѐҏӗѫӑ
24.06.2016
12:06:18
кстати, да
вместо веб эссембли надо

Alexander
24.06.2016
12:06:39
Хм, да, надо актуализировать
Морфдома не хватает ещё

Ҫѐҏӗѫӑ
24.06.2016
12:06:56
про галп я тоже не говорил такого вроде
он иногда нужен

anoru
24.06.2016
12:07:05
надо пуллреквесты кидать в футболку

Nikita
24.06.2016
12:07:20
и redux!

Alexander
24.06.2016
12:08:40
Да я просто иногда и в сарказм тоже учитывал )

Ҫѐҏӗѫӑ
24.06.2016
12:09:07
жс как ни крути добавить надо

Алексей
24.06.2016
12:23:33
https://habrahabr.ru/post/304000/

anoru
24.06.2016
12:26:18
очередной инферно?

localvoid
24.06.2016
12:26:53
он вроде появился раньше чем инферно :)

Andrey
24.06.2016
12:26:54

localvoid
24.06.2016
12:27:41

Алексей
24.06.2016
12:28:07

Vadim
24.06.2016
12:28:13
;)

Dmitry
24.06.2016
12:28:34
Нужен еще до того, как становится не нужен

Алексей
24.06.2016
12:28:56

Google

Dmitry
24.06.2016
12:29:02
:(
Не даешь похайпиться никому

Ҫѐҏӗѫӑ
24.06.2016
12:31:27
видом баян жи

Roman
24.06.2016
12:36:22
Чем Vidom отличается от React
- скорость, скорость и еще раз скорость
...
скорость - не нужна
скорость разработки - нужна

Kich
24.06.2016
12:39:13
А в head умеет рендерить?

anoru
24.06.2016
12:39:50
:D

from
24.06.2016
12:44:38
вроде нет, Костя молодец)
он через реакт context прокидывает методы
https://github.com/kriasoft/react-starter-kit/blob/master/src/components/App/App.js#L29-L33
и компоненты в cWM могут выставить title, meta и т.п.
Что на мой взгляд для компонентов-страниц норм решение.