Closed KoWeiJAC closed 1 month ago
Visit the preview URL for this PR (updated for commit cb92b0e):
https://jac-admin-develop--pr2390-feat-2387-improve-el-s0bfjq10.web.app
(expires Sun, 30 Jun 2024 08:39:41 GMT)
🔥 via Firebase Hosting GitHub Action 🌎
Sign: 4e92cf51659207b0ae3509dc5c40edde50edfec0
@KoWeiJAC I'm afraid you've misread this ticket: pls look again - for Statutory eligibility, we are asking to make a single recommendation for Statutory criteria, i.e. we need to display the flags for Prof Qualification and PQE but have only one recommendation dropdown and one reason free text field for the Statutory section.
Non-statutory section looks correct.
One other comment: is it possible to have a heavier line between candidates than we have within each candidate section?
I think this will help to pick out separate candidate entries.
@nickaddy Thank you for finding this incorrect place out. I've had adjustments accordingly. The statutory issues share one overall recommendation and reason. Also, I've made the line below candidate name heavier. Could you help me to review those places again ? Thanks!
@KoWeiJAC The changes look good - thank you for remedying. However, there is a strange error occurring: when you make changes to the RLoS section dropdown and/or free text fields, the same changes are made to the equivalent fields in statutory section; pls investigate. I am happy to demo if it's not happening for you.
@KoWeiJAC The changes look good - thank you for remedying. However, there is a strange error occurring: when you make changes to the RLoS section dropdown and/or free text fields, the same changes are made to the equivalent fields in statutory section; pls investigate. I am happy to demo if it's not happening for you.
@nickaddy Thank you for finding this bug, I've had a fix for it, could you help me to test it again? Thanks
@KoWeiJAC In reviewing the report download, I've noticed a couple of issues on this ticket:
View application
(to the right of candidates' names,) the page should open in a new tabReasons not satisfied
title of both free text fields should read JAC Comments
@KoWeiJAC Some more comments from last week's UT:
Display only candidates with Eligibility issues
tickbox?@KoWeiJAC Some more comments from last week's UT:
- The test exercise had no PQE eligibility set so how was it awarding Met flags to candidates? I’ve changed it to 5y and refreshed but the calculation of PQE does not seem to be working properly
- Related to the first point: Duration content in brackets missing after first candidate is missing
![]()
- We need to rethink this filter; it was designed for just one recommendation dropdown: could it include candidates where 1 or more of the recommendation dropdowns is unassigned?
![]()
- Once you have sorted the above, could you pls create an application that does meet all the criteria so that we can test the
Display only candidates with Eligibility issues
tickbox?
@nickaddy The refresh button is a little bit tricky, to prevent the system reset the recommendations, it only set one application for one time. It means if the application is added to report, this application won't be refreshed for not overwriting the existing recommendations. But the refresh button can add newly added applications to report. This mechanism is the same as the character issue report.
To improve it, maybe we can refresh(update) the application which have no recommendations. If the application have any recommendation being set, the refresh would skip this application for not overwriting those recommendations. Would you think it's workable ?
- The test exercise had no PQE eligibility set so how was it awarding Met flags to candidates?
@nickaddy The test exercise had no PQE eligibility set so how was it awarding Met flags to candidates? => The system will make the PQE to be Met as default if the PQE years is not set. Do we need to have change on this behaviour?
It looks like the only candidates that were flagged as Not Met were the ones that entered no work experience at all. If you check Exercise setup, the options under Eligibility Information are 5y, 7y or other - where you enter the number of years required. I don't understand how this exercise was created with 0 years PQE. If that is correct, even the candidates with no work experience should be flagged as Met.
It looks like the only candidates that were flagged as Not Met were the ones that entered no work experience at all. If you check Exercise setup, the options under Eligibility Information are 5y, 7y or other - where you enter the number of years required. I don't understand how this exercise was created with 0 years PQE. If that is correct, even the candidates with no work experience should be flagged as Met.
Just had a test, when creating the legal exercise, the PQE field is not required, so user can leave it empty and the PQE will be 0 as default. I was using this exercise for test: https://jac-admin-develop--pr2390-feat-2387-improve-el-s0bfjq10.web.app/exercise/l6idrKzbXLr7BdJxmZvk/details/eligibility
Not sure if all the legal exercises need to look on PQE? If so, maybe we can make the PQE field required when creating exercise. To ensure PQE is greater than 0 and the candidates with no experience will be fagged Not Met.
@KoWeiJAC I agree with your proposal above - the PQE field for legal exercises must be mandatory and any candidates that do not enter any experience flagged as Not Met (in addition to those that do not meet the required PQE.)
For reference, non-legal exercises do not request PQE according to my understanding.
@KoWeiJAC I agree with your proposal above - the PQE field for legal exercises must be mandatory and any candidates that do not enter any experience flagged as Not Met (in addition to those that do not meet the required PQE.)
For reference, non-legal exercises do not request PQE according to my understanding.
@nickaddy
@HalcyonJAC I can't find my comment, maybe it was in Slack, regarding the filter for unassigned, has this been fixed? The dropdown options should be All issues and Unassigned - the latter would show candidates where any of the 3 recommendations are not selected or any of the 3 free text fields not populated. Could you address the above pls?
@HalcyonJAC I can't find my comment, maybe it was in Slack, regarding the filter for unassigned, has this been fixed? The dropdown options should be All issues and Unassigned - the latter would show candidates where any of the 3 recommendations are not selected or any of the 3 free text fields not populated. Could you address the above pls?
@nickaddy I have fixed the filter. Could you retest it, please?
@HalcyonJAC Looking good, Ryan. Can you pls:
Generate report
button@nickaddy
What's included?
closed #2387
Who should test?
✅ Product owner ✅ Developers ✅ UTG
How to test?
Refresh
button to prepare eligibility issue data.Reasons not satisfied
box.Previous Judicial Experience
andReasonable length of service
issueRecommendation
andReasons not satisfied
changes can be saved.Refresh
button to prepare eligibility issue data.Risk - how likely is this to impact other areas?
🟢 No risk - this is a self-contained piece of work
Additional context
Include screen grabs, video demo, notes etc.
Related permissions
Have permissions been considered for this functionality?
PREVIEW:DEVELOP can be OFF, DEVELOP or STAGING