P5.js-web-editor: Promisify all mongoose queries using async/await

[catarak]

Nature of issue?

• [X] Existing feature enhancement

Feature enhancement details:

Right now, all of the mongoose queries use callbacks instead of the new ES6 promise syntax. This should be updated as nested callbacks are difficult to read.

[dhruvdutt]

I'm on it.

[himanshuc3]

@dhruvdutt still on it?

[dhruvdutt]

Yeah. I've already started with it. Will be pushing by this week.

[migom6]

is anyone assigned to this issue?

[catarak]

@dhruvdutt started working on it but i don't know how far they got. feel free to start on this!

[catarak]

it might be helpful to work on this in pieces, like promisify one controller at a time or something.

[dhruvdutt]

Ah! I'm sorry, couldn't get back to this. I would be happy to review a PR.
Best of luck @migom6 :+1:

[siddhant1]

Working!

[catarak]

this is a big issue, so i recommend making small PR's!

[shakeabi]

@catarak As the task is huge, I too would like to contribute to this.

[meiamsome]

Are we prefer the promise .then chains, or should the code be using async/await? I would prefer the latter for readability

[catarak]

i definitely prefer async / await! i'll update the issue title.

[siddhant1]

We have to change the error handling mechanics with async/await , I have some links , I will post them here

[catarak]

There's an in-depth conversation about some of the struggles of using async-await with Express: https://github.com/processing/p5.js-web-editor/pull/973

TL;DR It makes sense to include an async middleware like express-async-handler, or update to a different Node framework, like Fastify or Koa.