paf31 / purescript-thermite

A simple PureScript wrapper for React
MIT License
350 stars 55 forks source link

Update to purescript-aff 4 #90

Closed OndrejSlamecka closed 7 years ago

OndrejSlamecka commented 7 years ago

This PR updates the library to work with purescript-aff version 4. The only change required seems to be in a call to makeAff which now takes a single callback combining what was previously done in two, also the result of the Eff computation is now supposed to be a Canceler (https://pursuit.purescript.org/packages/purescript-aff/4.0.0/docs/Control.Monad.Aff#t:Canceler) -- I have used nonCanceler (does nothing) as I did not find anything that could be done at that point.

paf31 commented 7 years ago

Thanks!