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 fetch bill runs service #921

Merged
merged 5 commits into from
Apr 19, 2024
Merged

Conversation

Cruikshanks
Copy link
Member

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

Part of a series of changes to migrate the legacy view bill runs page into this project

In order to view the bill runs we need to fetch them first! This change adds a new service that generates a summary for each bill run for use in a view bill runs page we'll be adding later.

A key thing to note about this change is it is the first time we've added an Objection.js paging query. This service is going to support us adding pagination to our pages for the first time via the GOV.UK pagination component.

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

> Part of a series of changes to migrate the legacy view bill runs page into this project

In order to view the bill runs we need to fetch them first! This change adds a new service that generates a summary for each bill run for use in a view bill runs page we'll be adding later.

A key thing to note about this change is it is the first time we've added an [Objection.js paging query](https://vincit.github.io/objection.js/recipes/paging.html#paging). This service is going to support us adding pagination to our pages for the first time via the [GOV.UK pagination component](https://design-system.service.gov.uk/components/pagination/).
@Cruikshanks Cruikshanks added the enhancement New feature or request label Apr 18, 2024
@Cruikshanks Cruikshanks self-assigned this Apr 18, 2024
@Cruikshanks Cruikshanks marked this pull request as ready for review April 18, 2024 12:29
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.

@Cruikshanks Cruikshanks merged commit fce4db3 into main Apr 19, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the add-new-fetch-bill-runs-service branch April 19, 2024 15:08
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