Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

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

Open
chmnkh opened this issue Dec 25, 2019 · 2 comments
Open

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

chmnkh opened this issue Dec 25, 2019 · 2 comments

Comments

@chmnkh
Copy link
Contributor

chmnkh commented Dec 25, 2019

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

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

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

featureName: 'feature',
reducer: featureReducer,

вместо

reducers: { feature: featureReducer },

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

@Znack
Copy link
Contributor

Znack commented Dec 25, 2019

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

@sk1e
Copy link
Contributor

sk1e commented Dec 25, 2019

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

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants