tulip / ppe-logistics

Issue Tracking for PPELogistics.org
2 stars 1 forks source link

UIN - Create/Edit Customers and Create Orders #194

Open jackkrooss opened 4 years ago

jackkrooss commented 4 years ago

image

jackkrooss commented 4 years ago

https://docs.google.com/presentation/d/1QLCMbJh8hPtsxJZ11p6uNVNNFc_zr3KiR9e3ucT5SE8/edit#slide=id.g879235c161_0_5

ianslai commented 4 years ago

Here's what I have so far.

create edit orders

jackkrooss commented 4 years ago

Clear Form for the Organization does not reset the dropdown selector

jackkrooss commented 4 years ago

In the order manager is it easy to make the filter selections drop downs instead of text entry fields?

ianslai commented 4 years ago

Clear Form for the Organization does not reset the dropdown selector

Fixed

In the order manager is it easy to make the filter selections drop downs instead of text entry fields?

It's possible (I wouldn't call it easy). But I took a quick look at the MasksOn data and for each of those dropdowns, there would be about 5000 items in the dropdown (for both Organization and Customer filters). I'm not sure that having a dropdown there would necessarily be a good idea? Right now you can just type in a fragment of the names of either to filter for the one(s) you care about.

jackkrooss commented 4 years ago

Oh, didnt realize that you could enter fragment. That is fine then

On Mon, Jun 15, 2020, 4:42 PM ianslai notifications@github.com wrote:

Clear Form for the Organization does not reset the dropdown selector

Fixed

In the order manager is it easy to make the filter selections drop downs instead of text entry fields?

It's possible (I wouldn't call it easy). But I took a quick look at the MasksOn data and for each of those dropdowns, there would be about 5000 items in the dropdown (for both Organization and Customer filters). I'm not sure that having a dropdown there would necessarily be a good idea? Right now you can just type in a fragment of the names of either to filter for the one(s) you care about.

— You are receiving this because you authored the thread. Reply to this email directly, view it on GitHub https://github.com/tulip/ppe-logistics/issues/194#issuecomment-644446697, or unsubscribe https://github.com/notifications/unsubscribe-auth/APJASPPA5VEIFEWI3RXTGVDRW2WX3ANCNFSM4NNQIE3Q .

jackkrooss commented 4 years ago

Added clearer error message and an on screen instruction to make it clear you need to use Select button