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.

Sign up for GitHub

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

Jump to bottom

Manifest must have at least 1 transporter validation #468

Closed
Tracked by #106
dpgraham4401 opened this issue May 5, 2023 · 0 comments · Fixed by #479
Closed
Tracked by #106

Manifest must have at least 1 transporter validation #468

dpgraham4401 opened this issue May 5, 2023 · 0 comments · Fixed by #479
Assignees
@dpgraham4401
Labels
bug Something isn't working client Related to front end workings (React/Redux) good first issue Good for newcomers
Milestone
Create electronic...

Comments

@dpgraham4401
Copy link
Member

dpgraham4401 commented May 5, 2023 •
edited
Loading

Update the manifestSchema object, defined using the Zod library, so that the length of the transporters array must be greater than or equal to 1.

If a value is not present, an error message should be displayed under the path: ['transporters']

We stated this here

haztrak/client/src/components/Manifest/manifestSchema.ts

Line 154 in 196c1d1

.refine(
however did not implement it fully and it was causing problems.
@dpgraham4401 dpgraham4401 mentioned this issue May 5, 2023
Closed
18 tasks
@dpgraham4401 dpgraham4401 added bug Something isn't working good first issue Good for newcomers client Related to front end workings (React/Redux) and removed good first issue Good for newcomers labels May 5, 2023
@dpgraham4401 dpgraham4401 changed the title Manifest must have at least 1 transporter Manifest must have at least 1 transporter validation May 5, 2023
@dpgraham4401 dpgraham4401 added the good first issue Good for newcomers label May 9, 2023
@dpgraham4401 dpgraham4401 self-assigned this May 12, 2023
@dpgraham4401 dpgraham4401 mentioned this issue May 12, 2023
Merged
4 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@dpgraham4401 dpgraham4401

Labels
bug Something isn't working client Related to front end workings (React/Redux) good first issue Good for newcomers
Projects
None yet
Milestone
Create electronic manifests In e-Mani...
Development

Successfully merging a pull request may close this issue.

Transporter required validation dpgraham4401/haztrak
1 participant
@dpgraham4401