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

Use site type for handler options #591

Merged

Conversation

dpgraham4401
Copy link
Member

@dpgraham4401 dpgraham4401 commented Sep 25, 2023

Description

This PR limits the available roles a site can be selected for when creating a new manifest based on what that site's designation is in RCRAInfo. (e.g., a Tsdf can be a Generator but not vice versa).

We also did a little clean up of unused POC source.

Issue ticket number and link

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) server Related to backend workings (Django/Rest API) labels Sep 25, 2023
@dpgraham4401 dpgraham4401 merged commit 42f6744 into USEPA:main Sep 25, 2023
8 checks passed
@dpgraham4401 dpgraham4401 deleted the use_site_type_for_handler_options branch September 25, 2023 19:55
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) server Related to backend workings (Django/Rest API)
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

1 participant