Телеграмм чат группы reatom

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

источник

14:48пожаловаться #9

AW

Alex Wroud in ☃️ Reatom — стейт-менеджер

artalar

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

правильно если бы key использовался

источник

14:49пожаловаться #10

a

artalar in ☃️ Reatom — стейт-менеджер

Ок, попробую,исправлю.Или, если не сложно, PR welcome))

источник

14:50пожаловаться #11

AW

Alex Wroud in ☃️ Reatom — стейт-менеджер

если нужно указать строго, что ключ должен быть строкой, то [key: string] иначе писать [P in string]

источник

14:50пожаловаться #12

AW

Alex Wroud in ☃️ Reatom — стейт-менеджер

но даже так, я все же считаю неверным применение string тут

источник

14:51пожаловаться #13

SB

Sergey Belozyorcev in ☃️ Reatom — стейт-менеджер

@artalar наткнулся на undefined state :))))) (таску могу на себя взять)

Вот кейс.
https://github.com/artalar/reatom/issues/46

Само сообщение об ошибке должно в либе разруливаться.

А вот тайпинги, на возможный undefined не знаю... Реально ли через либу это разрулить? Или это на ответственности пользователя должно быть?

GitHub

Improve undefined state error · Issue #46 · artalar/reatom

const Dict = declareAtom('dict', {}, on => []); const getItemById = (id) => map(`item ${id}`, A, a => a[id]) store.getState(getItemById(10), state => {})...

источник

14:57пожаловаться #14

a

artalar in ☃️ Reatom — стейт-менеджер

С тайпингами можно попробовать что-то вроде State extends undefined ? never : State

источник

14:59пожаловаться #15

a

artalar in ☃️ Reatom — стейт-менеджер

Но это особо не поможет, потому что TC плохо проверяет коллекции разного рода

источник

15:00пожаловаться #16

AW

Alex Wroud in ☃️ Reatom — стейт-менеджер

блин я ; автоматом ставлю, а линтер не ругается )

источник

15:01пожаловаться #17

a

artalar in ☃️ Reatom — стейт-менеджер

Sergey Belozyorcev

@artalar наткнулся на undefined state :))))) (таску могу на себя взять)

Вот кейс.
https://github.com/artalar/reatom/issues/46

Само сообщение об ошибке должно в либе разруливаться.

А вот тайпинги, на возможный undefined не знаю... Реально ли через либу это разрулить? Или это на ответственности пользователя должно быть?

GitHub

Improve undefined state error · Issue #46 · artalar/reatom

const Dict = declareAtom('dict', {}, on => []); const getItemById = (id) => map(`item ${id}`, A, a => a[id]) store.getState(getItemById(10), state => {})...

Если таску сделаешь, будет гут

источник

15:02пожаловаться #18

SB

Sergey Belozyorcev in ☃️ Reatom — стейт-менеджер

Alex Wroud

блин я ; автоматом ставлю, а линтер не ругается )

Бывает. Я тоже не сразу привык :) А ещё с 4 на 2 символа )

источник