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

Two-part tariff due returns to show as blank #1198

Merged
merged 4 commits into from
Jul 22, 2024
Merged

Conversation

Beckyrose200
Copy link
Contributor

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

During testing for the two-part tariff review pages, it was raised that the billing and data team wanted to amend the returns displaying under a charge element. When a return matches a charge element, we show these under the review charge element section with their allocated quantity.
A due return currently displays 0 ML. The billing and data team wants to remove this and show a due return as blank instead of 0 ML. They say that showing 0 ML gives the false impression that the return is a nil return, rather than a due return. This PR is for this change.

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

During testing for the two-part tariff review pages, it was raised that the billing and data team wanted to amend the returns displaying under a charge element. Currently when a return matches to a charge element, we show these under the charge element with there allocated quantity. This means for a return that is due the quantity displayed is always 0. The billing and data team would like to remove this and show a due return as being blank instead of 0. They say that showing 0 gives the false impression that the return is a nil return, rather then a due return. This PR is for this change.
@Beckyrose200 Beckyrose200 added the enhancement New feature or request label Jul 19, 2024
@Beckyrose200 Beckyrose200 self-assigned this Jul 19, 2024
@Beckyrose200 Beckyrose200 marked this pull request as ready for review July 22, 2024 08:32
@Beckyrose200 Beckyrose200 merged commit d7c835e into main Jul 22, 2024
4 checks passed
@Beckyrose200 Beckyrose200 deleted the blank-returns-2pt branch July 22, 2024 09:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants