🚀🚀
вот такую
это можно в качестве пояснения к самому
attach({effect})Size: a a a
2020 October 06
🚀🚀
я про что-то типа таблицы в sample
🚀🚀
так как четыре режима
source, mapParams, effect
source, effect
mapParams, effect
effect
source, mapParams, effect
source, effect
mapParams, effect
effect
c
завел issue
c
ф
ID:0
effector patronum v0.109
- Теперь все методы имеют тесты на работу в fork, а значит поддерживают SSR
- Метод spread имеет типизацию и также покрыт тестами
- В методе combineEvents исправлена ошибка типизации
На шаг ближе к v1.0
- Теперь все методы имеют тесты на работу в fork, а значит поддерживают SSR
- Метод spread имеет типизацию и также покрыт тестами
- В методе combineEvents исправлена ошибка типизации
На шаг ближе к v1.0
я думаю, надо изменить дефолтное поведение combineEvents: чтобы после того как target (целевой ивент) триггернулся, то любой ивент из массива триггерил бы снова таргет с обновленным результатом - это концептуально более правильное поведение (триггерится когда доступны/накоплены данные из всех источников, например, для их дальнейшего комбинирования)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
c
я думаю, надо изменить дефолтное поведение combineEvents: чтобы после того как target (целевой ивент) триггернулся, то любой ивент из массива триггерил бы снова таргет с обновленным результатом - это концептуально более правильное поведение (триггерится когда доступны/накоплены данные из всех источников, например, для их дальнейшего комбинирования)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
запилишь? я тогда напилю migration guide и помечу везде, чтобы не забыть
DS
я сейчас понял, что возвращать из либы ивенты/эффекты, созданные внутри, которые предназначены для того, чтобы их запускали - это плохо в принципе...
вместо этого нужно чтобы пользователь сам создал иаенты/эффекты, которые он хочет запускать, и передал их в либу
вместо этого нужно чтобы пользователь сам создал иаенты/эффекты, которые он хочет запускать, и передал их в либу
Это касается не только либ , а любой модели очерченой доменной зоной
ф
запилишь? я тогда напилю migration guide и помечу везде, чтобы не забыть
постараюсь сделать pr, как смогу
c
так как четыре режима
source, mapParams, effect
source, effect
mapParams, effect
effect
source, mapParams, effect
source, effect
mapParams, effect
effect
видимо для каждого режима нужно описать motivation
и что-то вроде применимости, где полезно такое юзать
и что-то вроде применимости, где полезно такое юзать
ф
я думаю, надо изменить дефолтное поведение combineEvents: чтобы после того как target (целевой ивент) триггернулся, то любой ивент из массива триггерил бы снова таргет с обновленным результатом - это концептуально более правильное поведение (триггерится когда доступны/накоплены данные из всех источников, например, для их дальнейшего комбинирования)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
может кто-то, кто использует, захочет прокомментировать
🚀🚀
видимо для каждого режима нужно описать motivation
и что-то вроде применимости, где полезно такое юзать
и что-то вроде применимости, где полезно такое юзать
думаю да
🚀🚀
может кто-то, кто использует, захочет прокомментировать
ето так не работает)) изменение поведения всегда внезапное
c
я думаю, надо изменить дефолтное поведение combineEvents: чтобы после того как target (целевой ивент) триггернулся, то любой ивент из массива триггерил бы снова таргет с обновленным результатом - это концептуально более правильное поведение (триггерится когда доступны/накоплены данные из всех источников, например, для их дальнейшего комбинирования)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
а если нужно сделать repeat, как сейчас, то включать это опцией...
миграция простая: при переходе на 1.0 версию patronum нужно будет лишь включить явно эту опцию (если требуется)
может наоборот?
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
ф
может наоборот?
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
ты делал брейкинги и на 0.102, на v1.0 точно можно)
c
ты делал брейкинги и на 0.102, на v1.0 точно можно)
там были брейкинги которые явно ломали код
c
а тут будет неявно
то есть код не сломается с криками, а будет работать, но неправильно
то есть код не сломается с криками, а будет работать, но неправильно
ф
может наоборот?
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
если хочет без repeat, то передаешь опцию? Чтобы не ломать поведение
не хотелось бы... будет же какой-то общий migration guide? планируются ещё какие-то ломающие изменения?
🚀🚀
нужен какой-то термин для изменения которое хуже брейкинга
🚀🚀