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

Reducers dont support Multimethods #51

Closed
gavlooth opened this issue May 16, 2019 · 4 comments
Closed

Reducers dont support Multimethods #51

gavlooth opened this issue May 16, 2019 · 4 comments

Comments

@gavlooth
Copy link

Recently i had to do the following
https://i.imgur.com/DMGUjlt.png
The reason is that multimethods are not actually js functions but rather objects implementing the
Ifn protocol so in order to be used as reducers one has to call them from within an actual function
I think the same is true for keywords, sets and maps. It would be a good enhanchment to be able to use them as reducers without having a top level wrapper function

@lilactown
Copy link
Collaborator

Thanks for filing the issue! Should be fixed in 1c94760

@lilactown lilactown reopened this May 16, 2019
@lilactown
Copy link
Collaborator

I've published version 0.5.2. Can you give it a shot and see if it fixes this problem?

@gavlooth
Copy link
Author

Sure. I will ping you as soon as

@gavlooth
Copy link
Author

Works like a charm!!!

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

2 participants