-
Notifications
You must be signed in to change notification settings - Fork 1
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
LTD-5769: Add option to approve multiple cases #2294
base: dev
Are you sure you want to change the base?
Conversation
This is initially allowed to only certain team's queues so add a rule to enforce the same.
Success message not being displayed with templateview for some reason. It could possibly be because of the messages middleware as we handle post ourselves.
""" | ||
Submit approval recommendation for the selected cases | ||
""" | ||
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
To protect the view I think we need to test user permission here also similar to
lite-frontend/caseworker/cases/views/main.py
Line 515 in f20b5fd
if not rules.test_rule("can_licence_status_be_changed", self.request, self.licence): |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM also assume can't be merged until API has been merged.
Change description
Add button on cases queue view to approve multiple cases.
This is enabled selectively on specific teams queues and there is a rule to enforce this.
A new endpoint is added for this purpose and the relevant PR is uktrade/lite-api#2362