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

Do not display draft return versions in set up #1298

Merged
merged 2 commits into from
Aug 30, 2024

Conversation

Cruikshanks
Copy link
Member

https://eaflood.atlassian.net/browse/WATER-4407

Part of the work to replace NALD for handling return requirements

When we started displaying return versions in the view licence set up tab, we should have excluded those with a 'draft' status.

These have been imported from NALD but found to have no return requirements, which isn't valid in NALD.

Once the return requirements have gone live in WRLS, a decision will be made on what to do with them. Until then, we need to ensure they don't appear in this table.

https://eaflood.atlassian.net/browse/WATER-4407

> Part of the work to replace NALD for handling return requirements

When we [started displaying return versions](#1054) in the view licence set up tab we should have excluded those with a 'draft' status.

These have been imported from NALD, but found to have no return requirements which isn't valid in NALD.

A decision will be made on what to do with them once we return requirements has gone live in WRLS. Till then, we just need to ensure they don't appear in this table.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label Aug 30, 2024
@Cruikshanks Cruikshanks self-assigned this Aug 30, 2024
@Cruikshanks Cruikshanks marked this pull request as ready for review August 30, 2024 14:57
@Cruikshanks Cruikshanks merged commit 17f35fc into main Aug 30, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the exclude-draft-return-versions branch August 30, 2024 16:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants