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

Fix review licence page #902

Merged
merged 11 commits into from
Apr 16, 2024
Merged

Fix review licence page #902

merged 11 commits into from
Apr 16, 2024

Conversation

Beckyrose200
Copy link
Contributor

@Beckyrose200 Beckyrose200 commented Apr 11, 2024

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

Following on from testing the above ticket, an issue with the page was discovered where a piece of text is missing at the top of the page. The text is dynamically shown when there are charge elements that have a status of 'review'. This PR is to add this text.

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

Following from the testing of the above ticket, two issues with the page was discovered. The first being where we are meant to display the charge reference billable volume this is actually displaying the charge elements and the second issue being a piece of text missing at the top of the page. This PR is to fix those issues.
@Beckyrose200 Beckyrose200 added the bug Something isn't working label Apr 11, 2024
@Beckyrose200 Beckyrose200 self-assigned this Apr 11, 2024
@Beckyrose200 Beckyrose200 marked this pull request as ready for review April 11, 2024 15:27
@Beckyrose200 Beckyrose200 requested review from Jozzey and Cruikshanks and removed request for Jozzey April 11, 2024 15:27
Copy link
Contributor

@Jozzey Jozzey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Couple more changes following going through this with Alan

app/views/bill-runs/review-licence.njk Outdated Show resolved Hide resolved
@Beckyrose200 Beckyrose200 requested a review from Jozzey April 16, 2024 13:27
Copy link
Contributor

@Jozzey Jozzey left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@Beckyrose200 Beckyrose200 merged commit 2fcf993 into main Apr 16, 2024
6 checks passed
@Beckyrose200 Beckyrose200 deleted the fix-review-licence-page branch April 16, 2024 13:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants