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

TP2000-1503 Auto end-date measures #1345

Open
wants to merge 18 commits into
base: master
Choose a base branch
from

Conversation

mattjamc
Copy link
Collaborator

@mattjamc mattjamc commented Nov 28, 2024

TP2000-1503 Auto end-date measures

Why

When a commodity is end-dated, all related measures need to be end-dated. Currently tariff managers are doing this manually in the same workbasket and then moving the measure transactions to the top of the workbasket. This PR automates this process.

What

This PR adds functionality to automatically end measures for commodity codes which have been end-dated in a workbasket. A list view first displays all the measures and whether they will be end-dated or deleted (depending on whether they have started or not). Upon submission the task is passed to a Celery worker. Measures are end-dated or deleted and these transactions are automatically moved to the top of the workbasket.

List view
image

Confirmation page
image

List view once all measures have been ended
image

@mattjamc mattjamc marked this pull request as ready for review December 11, 2024 14:07
@mattjamc mattjamc requested a review from a team as a code owner December 11, 2024 14:07
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/util.py Outdated Show resolved Hide resolved
workbaskets/views/ui.py Show resolved Hide resolved
settings/common.py Outdated Show resolved Hide resolved
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants