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

BNGP-3724: Logic to check user status and paragraph suppression #637

Merged
merged 4 commits into from Feb 26, 2024

Conversation

thousand-leaves
Copy link
Collaborator

https://eaflood.atlassian.net/browse/BNGP-3724

For the allocations journey, the view for checking a users Defra account displays two paragraphs of text currently regardless of how questions are answered. This change enhances allocation journey logic to take account of whether the user is not acting on behalf of a client and is not the landowner/leaseholder (non-relevant person) and suppresses the second paragraph if so.

https://eaflood.atlassian.net/browse/BNGP-3724

For the allocations journey, the view for checking a users Defra account displays two paragraphs of text currently regardless of how questions are answered. This change enhances allocation journey logic to take account of whether the user is not acting on behalf of a client and is not the landowner/leaseholder (non-relevant person) and suppresses the second paragraph if so.
@thousand-leaves thousand-leaves self-assigned this Feb 16, 2024
@thousand-leaves thousand-leaves changed the title Logic to check user status and paragraph suppression BNGP-3724: Logic to check user status and paragraph suppression Feb 16, 2024
Copy link

sonarcloud bot commented Feb 22, 2024

@thousand-leaves thousand-leaves marked this pull request as ready for review February 22, 2024 12:44
@shaundefra shaundefra merged commit 9f2aece into master Feb 26, 2024
4 checks passed
@shaundefra shaundefra deleted the feature/BNGP-3724-non-relevant-person-check branch February 26, 2024 12:38
@shaundefra
Copy link
Collaborator

tested by Tajinder

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants