search

tonyvanriet / lunch-bot

A Slack bot that manages group lunch activity
MIT License
26 stars 5 forks source link

give a list of items for "ordered" command #8

Closed the-mikedavis closed 5 years ago

the-mikedavis commented 5 years ago

Closes #7.

the-mikedavis commented 5 years ago
Screen Shot 2019-07-15 at 09 06 30
tonyvanriet commented 5 years ago

@the-mikedavis this seems to assume that orders are always a single line. also, the intention was to be able to copy/paste the order command, but you took order out of the display. can't say I'm too keen on this PR in its current form.

"for" -> "from", top notch, love it.

the-mikedavis commented 5 years ago

ah right i’ve seen @bjornrud do some listicles in his order, should be multi line

I guess there are arguments for keeping the new line after ordered, especially the copy paste. but i’m looking to remove those arguments in #6 @tonyvanriet

the-mikedavis commented 5 years ago

Screen Shot 2019-07-16 at 12 00 34 PM