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

Scheduled status disabling #672

Merged
merged 8 commits into from
Jan 4, 2024

Conversation

dpgraham4401
Copy link
Member

@dpgraham4401 dpgraham4401 commented Jan 4, 2024

Description

This PR adjust the criteria for when the Scheduled option is available on the manifest form. It is displayed as disabled if ther user does not have access to the TSDF listed on the manifest (or no TSDF is added).

In addition, we've converted the manifest type to a react-select component for consistent style.

Issue ticket number and link

closes #565

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

This particular field gets a whole react component to itself since so much of the preceding manifest is dependent on its value
… manifest slice

manifest service module getStatusOptions business logic

fix 'enter generator info' button horizontal alignment

fix NewManifest auto applies site by site ID (from URL param)
and remaining bootstrap select components to react-select components for uniform look and feel
…ads and writes the current manifest status from the manifest slice

we will use this hook as an abstraction for talking to the redux store about the current manifest's status

In addition, I made these modifications:
    remove sync button from site details
@github-actions github-actions bot added CI/CD Requires modiciations to the continuous integration pipeline(s) client Related to front end workings (React/Redux) labels Jan 4, 2024
@dpgraham4401 dpgraham4401 merged commit 697b3c4 into USEPA:main Jan 4, 2024
5 checks passed
@dpgraham4401 dpgraham4401 deleted the scheduled_status_disabling branch January 4, 2024 20:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
CI/CD Requires modiciations to the continuous integration pipeline(s) client Related to front end workings (React/Redux)
Projects
No open projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Disable 'Scheduled' status option for non-TSDF users
1 participant