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

Fix cancel back link to PRESROC 2pt review #787

Merged
merged 2 commits into from
Mar 3, 2024

Conversation

Cruikshanks
Copy link
Member

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

To make cancelling bill runs more visible in the service we needed to re-implement the cancel bill run feature. Because we've also replaced all the legacy bill run pages we didn't think we'd need to deal with any legacy views.

That was until we spotted that the PRESROC two-part tariff review page is still alive and kicking in the service! You can cancel a bill run from that page so we must support going back to it in our confirm cancel bill run page.

This change makes that so.

@Cruikshanks Cruikshanks added the bug Something isn't working label Mar 3, 2024
@Cruikshanks Cruikshanks self-assigned this Mar 3, 2024
https://eaflood.atlassian.net/browse/WATER-4387

We spotted a little too late that the `app/views/bill-runs/cancel.njk` view is importing macros it is not using. We know this was because of a copy and paste and doesn't break anything.

Still, we don't like to make a habit of importing things and then not using them.
@Cruikshanks Cruikshanks force-pushed the fix-cancel-back-presroc-review branch from 1a17a33 to 21a191a Compare March 3, 2024 23:34
@Cruikshanks Cruikshanks marked this pull request as ready for review March 3, 2024 23:42
@Cruikshanks Cruikshanks merged commit 8ad9704 into main Mar 3, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the fix-cancel-back-presroc-review branch March 3, 2024 23:42
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant