react-redux-starter-kit icon indicating copy to clipboard operation
react-redux-starter-kit copied to clipboard

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

Open chmnkh opened this issue 6 years ago • 2 comments

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

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

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

featureName: 'feature',
reducer: featureReducer,

вместо

reducers: { feature: featureReducer },

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

chmnkh avatar Dec 25 '19 07:12 chmnkh

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

Znack avatar Dec 25 '19 09:12 Znack

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

sk1e avatar Dec 25 '19 10:12 sk1e