search

jac-uk / admin

The Vue SPA for managing applications and applicants
https://admin.judicialappointments.digital
MIT License
4 stars 2 forks source link

Feature/2419 Reference statuses in Independent Assessments task #2420

Closed HalcyonJAC closed 3 months ago

HalcyonJAC commented 3 months ago

What's included?

Closes #2419

Related PRs:

Who should test?

āœ… Product owner āœ… Developers āœ… UTG

How to test?

Example exercises:

Steps:

  1. Go to the Independent Assessments task.
  2. Check if the options and counts in the dropdown are generated based on the application statuses.

Risk - how likely is this to impact other areas?

šŸŸ¢ No risk - this is a self-contained piece of work

Additional context

Demo:

https://github.com/jac-uk/admin/assets/79906532/2ed7b9f8-b25c-479d-acec-15ffa3448ca9

Related permissions

Have permissions been considered for this functionality?

PREVIEW:DEVELOP can be OFF, DEVELOP or STAGING

github-actions[bot] commented 3 months ago

Visit the preview URL for this PR (updated for commit 1bff0e7):

https://jac-admin-develop--pr2420-feature-2419-referen-7tqgsgb3.web.app

(expires Thu, 18 Jul 2024 10:45:22 GMT)

šŸ”„ via Firebase Hosting GitHub Action šŸŒŽ

Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0

nickaddy commented 3 months ago

@HalcyonJAC I believe this works, however, I just had a thought: this may never happen, but if there was a case where there was a small exercise and IAs was the first task, i.e. no QT, no sift before, would the application status appear as blank in the dropdown? And, if so, would users be able to select and progress all candidates with a blank status?

HalcyonJAC commented 3 months ago

@nickaddy That case might happen indeed. For now, the blank status will not appear in the dropdown. I could make changes to support selecting candidates with a blank status if you would like.

nickaddy commented 3 months ago

@HalcyonJAC Better safe than sorry! Pls proceed with including the 'blank' option in the dropdown.

HalcyonJAC commented 3 months ago

@nickaddy The Blank option is included in the dropdown. Could you test it, please?