search

fullstack-development / react-redux-starter-kit

Modular starter kit for React+Redux+React Router projects.
https://demo.fullstack-development.com/
MIT License
91 stars 13 forks source link

Поменять интерфейс makeFeatureEntry #169

Open chmnkh opened 4 years ago

chmnkh commented 4 years ago

makeFeatureEntry полем reducers принимает объект, это нужно для того, чтобы можно было бы несколько редакс веток на фичу, + для того, чтобы использовать ключ как имя ветки фичи

во-первых, не понятно, что за "несколько веток на фичу", во-вторых, можно ключ для имени ветки фичи явно передавать

т.е. получится нечто такое:

featureName: 'feature',
reducer: featureReducer,

вместо

reducers: { feature: featureReducer },

+ не понятно зачем саги принимаются массивом, не понятно, зачем нужна такая гибкость, на худой конец там можно юнион принимать

Znack commented 4 years ago

@in19farkt @sk1e что думаете?

sk1e commented 4 years ago

API корявое немного, да. Эти проблемы будут решены в фабрике фич. У нас по плану 30-го встреча с обсуждением моих ишью. Думаю лучше подождать до этого времени и не обсуждать пока такие минорные заплатки.