Allow context to return a promise

[arafel]

I'm using a function to return a global context, and it needs to look up some things asynchronously. I found that Vision wasn't resolving the promise before continuing with render.

The change I've made fixes that, and the test suite still passes. I'm aware of #177, but that seems to be a wider issue (and has been going on for a while!). Please would you consider merging this?

Thanks!

[wswoodruff]

@arafel

Thank you for your work for this PR

Updating to async / await is a longstanding issue with vision because there's a lot of work involved, but we're working on it piecemeal starting with #205, which is a rebase of #177

Your message and PR reminded me we gotta get that settled and I pushed for merging it in the hapi hour Slack — hopefully it'll get merged soon!

[wswoodruff]

Are there any other vision functions you'd like to be async? We can collaborate on dividing up the work if you'd like to contribute

[arafel]

Thanks - I didn't know you could wait on concrete objects like that. Learning something new every day. :-)

I don't have anything else that I need async yet. Is the plan to wait and do the whole of vision in one go, or is it possible to async-ify it a bit at a time?

[wswoodruff]

We're going for it a bit at a time — starting with the issue you're addressing here — #205 just got merged! @hapi/vision v6.1.0 should be released to npm shortly 😄

[wswoodruff]

Alright v6.1.0 with async context support is now on npm does that solve this issue for you @arafel?

[arafel]

Hi @wswoodruff - I've just tested 6.1.0 and it works fine with my async context code. Much obliged, thank you. :-)

[Nargonath]

Thanks @wswoodruff for following that up.