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

Add new Compensation Charge Transaction presenter #506

Merged
merged 5 commits into from
Nov 6, 2023

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented Nov 6, 2023

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

Related to WATER-4155 and the work to replace the legacy bill page with our own we're now starting to build the page it will link to; a view of the bill licence and all its transactions.

This adds a presenter that given a TransactionModel instance whose charge type is compensation will return an object containing formatted details about the transaction.

The presenter handles both PRESROC and SROC transactions, altering the format of the output slightly by what needs to be displayed on the page when the transaction is displayed.

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

Related to WATER-4155 and the work to replace the legacy bill page with our own we're now starting to build the page it will link to; a view of the bill licence and all its transactions.

This adds a presenter that given a `TransactionModel` instance whose charge type is `minimum_charge` will return an object containing formatted details about the transaction.

The presenter handles both PRESROC and SROC transactions, altering the format of the output slightly in accordance with what needs to be displayed in the page when the transaction is displayed.
@Cruikshanks Cruikshanks added the enhancement New feature or request label Nov 6, 2023
@Cruikshanks Cruikshanks self-assigned this Nov 6, 2023
@Cruikshanks Cruikshanks marked this pull request as ready for review November 6, 2023 16:46
@Cruikshanks Cruikshanks merged commit 24832b3 into main Nov 6, 2023
6 checks passed
@Cruikshanks Cruikshanks deleted the add-compensation-charge-transaction-presenter branch November 6, 2023 20:46
Cruikshanks added a commit that referenced this pull request Nov 6, 2023
Related to WATER-4155 and the work to replace the legacy bill page with our own we're now starting to build the page it will link to; a view of the bill licence and all its transactions.

We added presenters for each of the transaction types

- [Minimum Charge Transaction presenter](#505)
- [Compensation Charge Transaction presenter](#506)
- [Standard Charge Transaction presenter](#507)

Now we add a presenter that brings those together with details about the bill and bill run.
Cruikshanks added a commit that referenced this pull request Nov 6, 2023
https://eaflood.atlassian.net/browse/WATER-4156

Related to WATER-4155 and the work to replace the legacy bill page with our own we're now starting to build the page it will link to; a view of the bill licence and all its transactions.

We added presenters for each of the transaction types

- [Minimum Charge Transaction presenter](#505)
- [Compensation Charge Transaction presenter](#506)
- [Standard Charge Transaction presenter](#507)

Now we add a presenter that brings those together with details about the bill and bill run.
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.

1 participant