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

Remove two-part tariff check endpoint from project #1002

Merged
merged 1 commit into from
May 12, 2024

Conversation

Cruikshanks
Copy link
Member

@Cruikshanks Cruikshanks commented May 12, 2024

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

To support the initial stages of building the new SROC two-part tariff match & allocate engine we created a /check endpoint. This would allow devs and testers alike to trigger the engine and review the results.

We are now sufficiently far along in the build of two-part tariff that we can perform the same checks using the review pages we've built. The focus has been on those pages for so long we'd need to change our /check endpoint to get it up to par!

It was only intended to be a temporary thing and now seems a good time to say goodbye.

So, thank you for your services to the cause /check! 🫡 😢

@Cruikshanks Cruikshanks added the housekeeping Refactoring, tidying up or other work which supports the project label May 12, 2024
@Cruikshanks Cruikshanks self-assigned this May 12, 2024
https://eaflood.atlassian.net/browse/WATER-4057

To support the initial stages of building the new SROC two-part tariff match & allocate engine we created a `/check` endpoint. This would allow devs and testers alike to trigger the engine and review the results.

We are now sufficiently far along in the build of two-part tariff that we can perform the same checks using the review pages we've built. In fact, the focus has been on those pages for so long we'd need to make changes to our `/check` endpoint to get it up to par!

It was only intended to be a temporary thing and now seems a good time to say goodbye.

So, hank you for your services to the cause `/check`! 🫡 😢
@Cruikshanks Cruikshanks force-pushed the delete-2pt-check-code branch from c9c299d to 2334efd Compare May 12, 2024 14:11
@Cruikshanks Cruikshanks marked this pull request as ready for review May 12, 2024 14:17
@Cruikshanks Cruikshanks merged commit 7430569 into main May 12, 2024
6 checks passed
@Cruikshanks Cruikshanks deleted the delete-2pt-check-code branch May 12, 2024 14:17
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