checkAPIAI is doing too much work

FAC10 / Genie-in-a-Bot

An chatbot for info on the UK General Elections

https://www.facebook.com/politicalgenie/

8 stars 2 forks source link

Open eliasmalik opened 7 years ago

eliasmalik commented 7 years ago

checkAPIAI.js is doing a lot of different things in one place. Logic for the following would ideally be separate:

parsing the event object into the inputs required by API.ai
sending the request to API.ai and dealing with potential request errors
scheduling tasks that need to run after the response comes back. this can be done with a combination of waterfall and parallel
dealing with any potential errors from the scheduled tasks

pbywater commented 7 years ago

@eliascodes Agreed - it's a mess! I'll try and clean it up 👍