Error function in effects "is not a function" about hyperapp HOT 7 CLOSED

Yeah the only way to see the error in my codepen example is to open upp the browser dev-tools. The code-pen console seems oblivious to anything but "console.log(...)"

I used code-pen just so I could provide a working example of the bug, but I'm happy to use any other tool you may suggest.

Thanks for pointing out that the docs are out of date!

from hyperapp.

@tunnckoCore jsbin is great for actual debugging 😄

But CodePen is nice to share things with. The console is weird is CodePen as well haha

from hyperapp.

Doc's are fine now, as far as I'm concerned at the moment anyway :) Nice swift work!

from hyperapp.

It comes from the docs. Third argument is data, fourth is the onError hook. app.js#L75

offtopic: I hate codepan.. how can i see the mentioned error? console shows nothing.
Never like it, never will - design, feel and all.

from hyperapp.

Thanks for pointing out that the docs are out of date!

@zaceno thanks, we didn't notice that.

from hyperapp.

@zaceno Thanks for reporting, gonna fix those doc-doc-docs!

Correct signature should be:

waitThenAdd: (model, msg, data, error)

For the record, if you update the pen as follows, here's what happens.

const effects = {
    waitThenAdd: (model, msg, data, error) => {
        console.log(data) // => HTML event since onclick=${msg.waitThenAdd}
        msg.toggle()
        wait(1000)
          .then(msg.add)
          .then(msg.toggle)
          .then(() => {error('just a message')}) // => throws the error with the specified message
    }
}

from hyperapp.

@zaceno Fixed, see https://github.com/hyperapp/hyperapp#effects

Let me know if that looks good to you. I think we're done here, so I'll close, but feel free to reopen if you feel this is still incomplete.

/cc @tunnckoCore

from hyperapp.