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

CAPT-1922 admin full task view #3382

Merged
merged 3 commits into from
Nov 13, 2024
Merged

Conversation

rjlynch
Copy link
Contributor

@rjlynch rjlynch commented Nov 8, 2024

Adds the missing tasks for the EY admin claims along with some changes to the
code to support this. May be easier to review each commit separatley.

@rjlynch rjlynch force-pushed the CAPT-1922/admin-full-task-view branch 2 times, most recently from f1d2aff to 43ac396 Compare November 8, 2024 14:24
@rjlynch rjlynch added the deploy Deploy a review app for this PR label Nov 8, 2024
@rjlynch rjlynch marked this pull request as ready for review November 8, 2024 15:29
Some policies will have conditions around when a task is available. This
commit adds a new method to the base policy `task_available?` which can
be overwritten by sub policies. We also update the tasks views to show a
policy specific message when the task is not available.
Adds the remaining EY tasks along with copy for the tasks being
unavilable.
We no longer need this method as we now check the policy rather than the
task when rendering the admin view.
@rjlynch rjlynch force-pushed the CAPT-1922/admin-full-task-view branch from f44564b to 9210943 Compare November 13, 2024 14:12
@rjlynch rjlynch merged commit af6d695 into master Nov 13, 2024
14 checks passed
@rjlynch rjlynch deleted the CAPT-1922/admin-full-task-view branch November 13, 2024 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
deploy Deploy a review app for this PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants