-
Notifications
You must be signed in to change notification settings - Fork 0
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
Amend billable volume on element review page #878
Conversation
https://eaflood.atlassian.net/browse/WATER-4370 As part of our two-part tariff bill run work, we've developed licence review pages where you can view the matching details of the two-part tariff engine. When viewing a charge elements matching details you have the option as a user to be able to change the billable volume on the element. This PR covers that functionality.
SonarCloud is currently complaining that there is duplicate code coverage within the amend-billable-returns-presenter. This is valid as there is a lot of duplicated code in the presenters but can be ignored for the time being. The plan is once all the pages have been built, to then go back into all the presenters and services and see what can be de-duplicated. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Just a couple of comments
test/presenters/bill-runs/two-part-tariff/amend-billable-returns.presenter.js
Outdated
Show resolved
Hide resolved
test/services/bill-runs/two-part-tariff/submit-amended-billable-returns.service.test.js
Show resolved
Hide resolved
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
https://eaflood.atlassian.net/browse/WATER-4370
As part of our two-part tariff bill run work, we've developed licence review pages where you can view the matching details of the two-part tariff engine. When viewing a charge elements matching details you have the option as a user to be able to change the billable volume on the element. This PR covers that functionality.