siddharthgutta / Entree.LegacyServer

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

Give user order id and instructions #352

Closed jlmao closed 8 years ago

jlmao commented 8 years ago

Displays order id to user and tells them to show name and id when picking up order 12992004_10153547336586149_2086710523_o

mathew-kurian commented 8 years ago

Can you please add a reference to the issue in these pull requests? It is hard to know which issue you are talking about. Also, some indication whether it is a new feature or a bugfix (which can be done with the branch name) or you some other way you like @jlmao @jadesym

jadesym commented 8 years ago

@jlmao I already made an individual issue for each of the items on the round 4 bugfix make sure to mark this one as done as well when this is pushed in https://github.com/siddharthgutta/Entree.Server/issues/342

mathew-kurian commented 8 years ago

@jlmao "to be picked up!" part seems repetitious since you already have "you arrive." Could you potentially one without the other?

jadesym commented 8 years ago

@bluejamesbond I think the the text he has is perfect. In this case, it's better to be very clear, since it may not be at what point in time or where you are supposed to present those things. We've made the mistake of giving people the benefit of the doubt in our chatbot, and I also don't think the extra 3 words negatively affect the experience especially since they improve clarity.