IBM-Cloud / logistics-wizard-webui

Web UI for Logistics Wizard Showcase demo. The Logistics Wizard is an end-to-end, smart supply chain management solution that showcases how to execute hybrid cloud, microservices, and predictive data analytics in the real world.
Other
14 stars 20 forks source link

Create UI for "CreateNewUser" action in role switcher #47

Closed colbycheeze closed 8 years ago

colbycheeze commented 8 years ago

No longer valid with new create demo/login design (it's done automatically)

l2fprod commented 8 years ago

@colbycheeze

I would go for a dialog asking "Name" (that we could prefill) and selecting one of the available retail storage (one without a manager set):

image

l2fprod commented 8 years ago

from @colbycheeze

I think it would be easier to just have like 20-30 random names it could pull from, and instead of a dropdown…have a select list / radios…since there are only a few locations (design best practices are to stay away from dropdowns unless there is no other option) so it could just be as simple as “Choose a location” and then a list of locations, then when you click one…it creates it

l2fprod commented 8 years ago

https://github.com/IBM-Bluemix/logistics-wizard-webui/tree/create-retailer

colbycheeze commented 8 years ago

Spoke with Ram about just having all 5 retail managers created in tandem with demo session creation. There doesn't seem to be any benefit to a tech seller being able to manually them up theirselves.

Correct me if I'm wrong.

This would eliminate the need to build all of this, and instead I'd just add the api calls alongside the demo creation saga code.

l2fprod commented 8 years ago

@colbycheeze @rvennam

Makes sense.

In that case, let's handle this on the ERP API side when we call POST /Demos. In the ERP code, in common/models/demo.js, Demo.newDemo creates the demo, seeds with data and creates the supply manager. We can create the retailer store managers there and assign them to the stores.

We would also remove the /Demos/{guid}/createUser method from the ERP API (and controller side too).

colbycheeze commented 8 years ago

That would be great. Would make the front end code a lot easier as well.