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

New box form with the option of adding samples from inventory #1731

Closed
diegoliberman opened this issue Jun 22, 2022 — with Manas.Tech Commit · 3 comments · Fixed by #1825
Closed

New box form with the option of adding samples from inventory #1731

diegoliberman opened this issue Jun 22, 2022 — with Manas.Tech Commit · 3 comments · Fixed by #1825
Assignees

Comments

Copy link
Contributor

diegoliberman commented Jun 22, 2022

The new box form should have the following fields:

  1. Box ID
  2. Purpose
  3. Media
  4. Samples source. New field, radio button with 2 options: 1) Create new samples; 2) Select samples from inventory

When the user selects ... from inventory, we need to show the Content field with the Add sample feature. Today, this is shown when the user picks the "Other" purpose.

When the user selects Create new samples let's do nothing, it's out of the scope of this ticket.

Acceptance criteria

  1. I can select the option Pick from inventory and I see the Add sample feature appearing.
  2. I can add 1 to N samples.
  3. I can save the box and see it appearing in the list of boxes.
  4. I can transfer the box with all its samples.

Invision

  1. Select samples from inventory
  2. Add samples
  3. List of added samples
@omelao
Copy link
Contributor

omelao commented Nov 9, 2022

What about the Save & Print button of these screens? Is there any ticket to make it work?

@diegoliberman
Copy link
Contributor Author

Good point! I created #1786. Thanks Carlos!

@sardar-usman
Copy link

@omelao Working fine as expected.

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