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

Manifest type option availability #674

Merged

Conversation

dpgraham4401
Copy link
Member

@dpgraham4401 dpgraham4401 commented Jan 4, 2024

Description

This PR adjust the manifest type select input. The new component

  1. has Hybrid type as the default
  2. FullElectronic option is disabled if a generator has not been added to the manifest
  3. FullElectronic option is disabled if the selected generator cannot e-Sign
  4. has unit tests associated with the component.

Issue ticket number and link

closes #673

Checklist

  • I have added tests that prove my fix is effective or that my feature works
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • My changes generate no new warnings

@github-actions github-actions bot added client Related to front end workings (React/Redux) dependencies project dependency problems or modifications labels Jan 4, 2024
@dpgraham4401 dpgraham4401 force-pushed the manifest_type_option_availability branch from 6e8edc7 to 1ebeb4a Compare January 4, 2024 22:57
@dpgraham4401 dpgraham4401 merged commit fc2ae37 into USEPA:main Jan 4, 2024
5 checks passed
@dpgraham4401 dpgraham4401 deleted the manifest_type_option_availability branch January 4, 2024 22:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
client Related to front end workings (React/Redux) dependencies project dependency problems or modifications
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Electronic manifest type filtering
1 participant