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

Distribution with Units from Request - UX Improvement #4604

Closed
awwaiid opened this issue Aug 19, 2024 · 7 comments · Fixed by #4651
Closed

Distribution with Units from Request - UX Improvement #4604

awwaiid opened this issue Aug 19, 2024 · 7 comments · Fixed by #4651

Comments

@awwaiid
Copy link
Collaborator

awwaiid commented Aug 19, 2024

When a Bank Organization is fulfilling a request that has units, they are prompted with the units requested:

image

However, it is not obvious on this screen that the number they enter needs to be singular Units, not packs or flats or anything. I think there should be something in the UI that reminds the bank user that they are entering singular units.

@cielf
Copy link
Collaborator

cielf commented Aug 20, 2024

Hmm. First suggestion-- add Quantity (Total Units) as a heading over the quantity? Maybe with the handholdy part of that only showing if it's an org with packs?

@cielf
Copy link
Collaborator

cielf commented Sep 1, 2024

Confirmed -- Add Quantity (Total Units) as a heading over the quantity.

@cielf cielf added the Help Wanted Groomed + open to all! label Sep 1, 2024
@Aaryanpal
Copy link
Contributor

@cielf please assign this.

@github-actions github-actions bot removed the Help Wanted Groomed + open to all! label Sep 12, 2024
@cielf
Copy link
Collaborator

cielf commented Sep 12, 2024

You've got it!

@Aaryanpal
Copy link
Contributor

image

@cielf will this works?

@dorner
Copy link
Collaborator

dorner commented Sep 13, 2024

I think the word "Add" is not supposed to be there.

@cielf
Copy link
Collaborator

cielf commented Sep 13, 2024

That's right -- I meant to say that we needed to add "Quantity - Total Units".

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

Successfully merging a pull request may close this issue.

4 participants