search

f-o-a-m / purescript-web3

a purescript library for the web3 api
Apache License 2.0
127 stars 24 forks source link

Created EventHandler type #40

Closed kejace closed 6 years ago

kejace commented 6 years ago

To merge or not to merge, that's the question ¯\(ツ)/¯

XertroV commented 6 years ago

It's hard to see what you've actually done here due to your changes in formatting.

Have you just refactored (a -> ReaderT Change (Web3 p e) EventAction) to be aliased as EventHandler a?

Was there any good reason for changing the formatting like you did?

martyall commented 6 years ago

I'm going to go ahead and close this, something else is on the way.