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

@bem-react/di proposal for fill from object #452

Closed
belozer opened this issue Jul 31, 2019 · 1 comment
Closed

@bem-react/di proposal for fill from object #452

belozer opened this issue Jul 31, 2019 · 1 comment

Comments

@belozer
Copy link
Member

belozer commented Jul 31, 2019

Сейчас приходится писать такую штуку, дублируя имена в аргументах

registry
    .set('PriceDiscount', PriceDiscount)
    .set('PriceVal', PriceVal)
    .set('Currency', Currency)

Но кажется, что заполнение реестра через объекты делать попроще.
Например можно сделать метод fill для таких вещей

registry.fill({
    PriceDicount,
    PriceVal,
    Curency
});

Или есть проблемы с типизацией этого подхода?

@belozer belozer changed the title @bem-react/di заполнение через объект @bem-react/di proposal for fill from object Jul 31, 2019
@belozer
Copy link
Member Author

belozer commented Aug 12, 2019

Более удачный подход реализован в PR #462, эту issue пока закрою.

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

No branches or pull requests

2 participants