🚀🚀
бери пока оба
я пока fork улучшаю, пофиксил заполнение values, добавил поддержку handlers, сейчас чейнджлог пишу)
Size: a a a
2020 June 07
c
чур в forest будет choice 😄
переименуешь variant?))
R
я пока fork улучшаю, пофиксил заполнение values, добавил поддержку handlers, сейчас чейнджлог пишу)
🚀🚀
https://github.com/zerobias/effector/blob/master/packages/effector/index.d.ts#L169
еще ts ругается, что у стора sid
а в форке только стринг https://github.com/zerobias/effector/blob/master/packages/effector/fork.d.ts#L5
еще ts ругается, что у стора sid
string | nullа в форке только стринг https://github.com/zerobias/effector/blob/master/packages/effector/fork.d.ts#L5
короче проще всего вот так юзать https://github.com/zerobias/effector/blob/master/src/types/__tests__/effector/fork.test.ts#L128
🚀🚀
так как null в поля объекта тайпскрипт класть не даёт, а у юнитов значение sid действительно string | null
R
короче проще всего вот так юзать https://github.com/zerobias/effector/blob/master/src/types/__tests__/effector/fork.test.ts#L128
варика через Map больше не будет?
🚀🚀
будет, просто он тут как мне кажется проще не делает https://github.com/zerobias/effector/blob/master/src/types/__tests__/effector/fork.test.ts#L115
🚀🚀
более того, handlers точно так же поддерживает и объекты и мапы, ну, вдруг найдётся способ подружить их с типами)
R
будет, просто он тут как мне кажется проще не делает https://github.com/zerobias/effector/blob/master/src/types/__tests__/effector/fork.test.ts#L115
скорее мне приятно знать, что вся магия происходит под капотом :) при использовании Map
🚀🚀
я там кстати убрал необходимость ещё в одной магии)
R
более того, handlers точно так же поддерживает и объекты и мапы, ну, вдруг найдётся способ подружить их с типами)
использование handlers в тестах уже есть?
🚀🚀
использование handlers в тестах уже есть?
R
уфф, когда выкатишь?)
🚀🚀
пять минут назад
🚀🚀
R
я просто уже давно думал о том, что variant не совсем корректное название, так как «вариант» — это «один из», а не сам метод, относящийся ко всем вариантам
вообще должен быть глагол
🚀🚀
не факт, наречие по идее тоже можно)
R
R
тем временем в эффекторе