Skip to content

Conversation

@daniel-manta-hee
Copy link
Contributor

Add flagging of competencies and display in supervisor and self assessment views.

JIRA link

https://hee-dls.atlassian.net/browse/DLSV2-523

Description

Added migration, updated views, controllers and tests.

DB model flags

FrameworkFlags

Screenshots

Framework Details tab
https://localhost:44363/Framework/2072/Details

image

Framework custom flags
https://localhost:44363/Frameworks/Flags/2072

image

Add custom flag
https://localhost:44363/Frameworks/Flags/New/2072

image

Edit custom flag
https://localhost:44363/Frameworks/Flags/Edit/2072/1012

image

Add/Edit custom flag validation

image

Remove Custom flag
https://localhost:44363/Frameworks/Flags/Delete/2072/1012

image

Edit competency
https://localhost:44363/Frameworks/2072/Competency/1167/1743

image

Framework competency preview
https://localhost:44363/Frameworks/2072/Competency/1167/1743/Preview

image

Framework structure interface
https://localhost:44363/Framework/2072/Structure/1167#fc-1743

image

Self assessment interface
https://localhost:44363/LearningPortal/SelfAssessment/1/Capabilities

image

Self assessment competency view/review
https://localhost:44363/LearningPortal/SelfAssessment/1/1

image

Supervisor interface
https://localhost:44363/Supervisor/Staff/2007/ProfileAssessment/1/Review

image


Developer checks

I have checked:

  • Adding, editing and removing flags to different capabilities and frameworks
  • Form validation is in place

@kevwhitt-hee kevwhitt-hee changed the title Add flagging of competencies and display in supervisor and self assessment views DLSV2-523 Add flagging of competencies and display in supervisor and self assessment views Jun 16, 2022
Copy link
Contributor

@kevwhitt-hee kevwhitt-hee left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This looks fantastic. Thanks Daniel. One very minor change requested - see code comments.

<tr role="row" class="nhsuk-table__row first-row">
<td role="cell" rowspan="@competency.AssessmentQuestions.Count()" class="nhsuk-table__cell">
<td role="cell" rowspan="@competency.AssessmentQuestions.Count()" class="nhsuk-table__cell">
<partial name="_ComperencyFlags" model="competency.CompetencyFlags" />
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

When rendered here, the flag container could do with nhsuk-u-padding-left-4 class adding so that they align with the competency beneath them, which is indented from the left:
image

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Added new commit.

…escription set to false, so they look in line with other items grouped under same flags
@kevwhitt-hee kevwhitt-hee merged commit 79aa1e2 into master Jun 21, 2022
@kevwhitt-hee kevwhitt-hee deleted the Develop/Features/DLSV2-523-FlaggingOfCompetencies branch June 21, 2022 10:07
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.

3 participants