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

Rename bill presenters to match convention #538

Merged
merged 1 commit into from
Nov 22, 2023

Conversation

Cruikshanks
Copy link
Member

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

We've realised that the presenters we built to support the new view bill, and bill licence pages don't follow the convention we've been following. Principally, we start with an adjective. For example, view or create. We do this to give new maintainers a sense of what context the presenter is used in.

So, this is a housekeeping change to rename the presenters and bring them in line.

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

We've realised that the presenters we built to support the new view bill, and bill licence pages don't follow the convention we've been following. Principally, that we start with an adjective. For example, `view` or `create`. We do this to give new maintainers a sense of what context the presenter is used in.

So, this is a housekeeping change to rename the presenters and bring them inline.
@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label Nov 22, 2023
@Cruikshanks Cruikshanks self-assigned this Nov 22, 2023
@Cruikshanks Cruikshanks marked this pull request as ready for review November 22, 2023 22:52
@Cruikshanks Cruikshanks merged commit 00e383f into main Nov 22, 2023
6 checks passed
@Cruikshanks Cruikshanks deleted the rename-existing-bill-presenters branch November 22, 2023 22:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
housekeeping Refactoring, tidying up or other work which supports the project
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant