siddharthgutta / Entree.LegacyServer

SMS-based Order Ahead Platform
https://textentree.com
2 stars 0 forks source link

Round 2 Pre-Launch Bug Fixes #253

Open jadesym opened 8 years ago

jadesym commented 8 years ago

Chatbot

mathew-kurian commented 8 years ago

@jadesym Double Text when enter in credit card info NECESSARY. Can you take a look at this please?

mathew-kurian commented 8 years ago

Model Attribute Request

@jlmao in the next change for the db, can you please add attribute Restaurant.profileImage which will contain a url to display the image

mathew-kurian commented 8 years ago

@jlmao Details about the aggregation bug

mathew-kurian commented 8 years ago

@jlmao should we be logging this? What does this error mean?

jadesym commented 8 years ago

@bluejamesbond double text just means you need to remove the message in all places:

You order has been sent. Hang tight!

mathew-kurian commented 8 years ago

@jadesym Oh ok my bad. I meant this

If profile update fails if an incorrect CC number, it will still fail even if CC number is corrected CHECK THIS
mathew-kurian commented 8 years ago

@jlmao Order updates. Not sure if they are working. Referring to this code base. Although the order is getting updated in the enterprise app via Sockets, I am not sure why the texts aren't going out.

The event is being emitted from here:

jadesym commented 8 years ago

@bluejamesbond check the bug fixes that you've completed