Skip to content

Usability - Create a user with machine attached errors out #3

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

Open
adanaie opened this issue Feb 6, 2013 · 2 comments
Open

Usability - Create a user with machine attached errors out #3

adanaie opened this issue Feb 6, 2013 · 2 comments

Comments

@adanaie
Copy link
Contributor

adanaie commented Feb 6, 2013

Should make it so user can be added with a machine attached, or change the UI so it progresses through naturally

@ghost ghost assigned EndymionJkb Feb 6, 2013
@EndymionJkb
Copy link
Contributor

I had it that way initially, but it was a separate screen. Let me see if I can make it "see" the in-memory ones that haven't been saved yet. If that works it might be ok. The cleanest is to create machines and users, then go to another page to assign them.

@EndymionJkb
Copy link
Contributor

OK; I had to redo it for two reasons. Not only was it clunky, there was a real problem that blocked you from assigning at the same time anyway. New users aren't made into technicians until they're saved, and you can't assign people to machines who aren't technicians. So I removed the assignment from the main edit screen, and added logic to detect whether any machines or users have been changed. If you just edit the address or phone or something, it updates and goes to the show view. If you've added or removed users, though it takes you to an assignment screen.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants