update view API to be (state, prev, send)

Since v2.1.4 (oops, semver) we’re now also passing the oldState down to views. The view api now looks like:

const view = (params, state, send, oldState) => choo.view`
  <main>${params.foo} and ${state.bar}</main>
`

Instead I reckon it might be nicer to have:

const view = (state, oldState, send) => choo.view`
  <main>${state.params.foo} and ${state.bar}</main>
`

This would definitely be a breaking change, but actually not that hard to fix (e.g. we can provide a regex heyyyy). What do people think?

[yoshuawuyts]

@timwis yeah, you’re totally right - mentally I’ve been geared to have data go first and any type of callback last, so we should probs just do (state, prev, send) ✨

[yocontra]

// view(context, send)
const view = ({ params, state, oldState }, send) => choo.view`
  <main>${params.foo} and ${state.bar}</main>
`

🙏