Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Connect Robot with Biz #108

Merged
merged 7 commits into from
Nov 8, 2018
Merged

Connect Robot with Biz #108

merged 7 commits into from
Nov 8, 2018

Conversation

squishykid
Copy link
Member

@squishykid squishykid commented Nov 7, 2018

  • Upload image during robot flow (currently a dodgy file)
  • make a reservation
  • show which tables are actually available

@squishykid squishykid requested review from a team as code owners November 7, 2018 04:58
@squishykid squishykid mentioned this pull request Nov 7, 2018
4 tasks
@arosspope
Copy link
Member

Looks pretty good - should we bother checking that the user has Robot privileges?

@squishykid
Copy link
Member Author

Yes, we should make an issue for that. That's low priority though.

arosspope
arosspope previously approved these changes Nov 7, 2018
Copy link
Member

@arosspope arosspope left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@squishykid
Copy link
Member Author

Doesn't look good to Trav.

@squishykid squishykid self-assigned this Nov 8, 2018
@squishykid squishykid added this to the End of Build 2 milestone Nov 8, 2018
@squishykid squishykid added review please Please review ready to merge ready to merge labels Nov 8, 2018
@jjbdlsantos jjbdlsantos self-requested a review November 8, 2018 13:49
Copy link
Contributor

@jjbdlsantos jjbdlsantos left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

it work on bigmeme wao

@squishykid squishykid merged commit 32c3530 into master Nov 8, 2018
@squishykid squishykid deleted the robot-biz branch November 8, 2018 13:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants