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

Creating new tables (Manager creation instead of selection) #15

Open
Gabetn opened this issue Mar 19, 2019 · 2 comments
Open

Creating new tables (Manager creation instead of selection) #15

Gabetn opened this issue Mar 19, 2019 · 2 comments
Labels
wontfix This will not be worked on

Comments

@Gabetn
Copy link
Collaborator

Gabetn commented Mar 19, 2019

Currently manager selects table# for device from dropdown, but it is limited by the DatabaseLoader file. It is unclear from our user stories if it is necessary for us to allow the manager to create a new table in that list.

See #12 form more information

This is low-priority as it is a simple task. We need to decide on what to do.

@Gabetn Gabetn added the wontfix This will not be worked on label Mar 19, 2019
@adeeb27
Copy link
Owner

adeeb27 commented Mar 19, 2019

Do we even need a drop-down menu? Why not just make a text input, where the manager just types in a number.

@Gabetn
Copy link
Collaborator Author

Gabetn commented Mar 19, 2019

Well, doing it via text input would require:

  1. validation of input (make sure it's not already assigned).
  2. creation of dining sessions inside the db in the case it's a new number

Whereas doing it via drop-down:

  1. no need for validation since we can just display only the unassigned (which I already did)
  2. creation of dining sessions is the thing I was unsure about us needing to do

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
wontfix This will not be worked on
Projects
None yet
Development

No branches or pull requests

2 participants