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 Minimum Charge Transaction presenter #505

Merged
merged 3 commits into from
Nov 6, 2023

Conversation

Cruikshanks
Copy link
Member

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 doesn't need to make any distinction between PRESROC and SROC transactions as they are both displayed in the same way.

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 doesn't need to make any distinction between PRESROC and SROC transactions as they are both displayed in the same way.
@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:07
Copy link
Contributor

@Beckyrose200 Beckyrose200 left a comment

Choose a reason for hiding this comment

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

@Cruikshanks Cruikshanks merged commit 34554b8 into main Nov 6, 2023
4 checks passed
@Cruikshanks Cruikshanks deleted the add-minimum-charge-transaction-presenter branch November 6, 2023 20:37
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.

2 participants