Skip to content

Conversation

@sherif-olaboye
Copy link
Contributor

@sherif-olaboye sherif-olaboye commented Sep 25, 2025

JIRA link

https://hee-tis.atlassian.net/browse/TD-5563

Description

Added ConfirmChangePrimaryFramework view to let users confirm make primary framework and add features to the framework.
Implemented GET and POST methods in ConfirmChangePrimaryFramework to render and submit the confirmation.
Added a Change Primary button to the SelectFrameworkSources view, enabling users to change their primary framework.

Screenshots

image image image

Developer checks

(Leave tasks unticked if they haven't been appropriate for your ticket.)

I have:

  • Run the IDE auto formatter on all files I’ve worked on and made sure there are no IDE errors relating to them
  • Written or updated tests for the changes (accessibility ui tests for views, tests for controller, data services, services, view models created or modified) and made sure all tests are passing
  • Manually tested my work with and without JavaScript (adding notes where functionality requires JavaScript)
  • Tested any Views or partials created or changed with Wave Chrome plugin. Addressed any valid accessibility issues and documented any invalid errors
  • Updated my Jira ticket with testing notes, including information about other parts of the system that were touched as part of the MR and need to be tested to ensure nothing is broken
  • Scanned over my pull request in GitHub and addressed any warnings from the GitHub Build and Test checks in the GitHub PR ‘Files Changed’ tab
    Either:
  • Documented my work in Confluence, updating any business rules applied or modified. Updated GitHub readme/documentation for the repository if appropriate. List of documentation links added/changed:
  • Confirmed that none of the work that I have undertaken requires any updates to documentation

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 good, so far @sherif-olaboye. It looks like we are missing an implementation of the following requirement from the ticket:

We should allow the user to change the primary framework for their self assessment. When doing so, we should prompt them to ask which features of the new primary framework should be applied to the self assessment (using a similar interface to that implemented in TD-5562: Prompt users to ask which features of the framework they want to copy to the assessment

Base automatically changed from DLS-Release-v1.3.0 to DEV September 25, 2025 16:07
@sherif-olaboye
Copy link
Contributor Author

This looks good, so far @sherif-olaboye. It looks like we are missing an implementation of the following requirement from the ticket:

We should allow the user to change the primary framework for their self assessment. When doing so, we should prompt them to ask which features of the new primary framework should be applied to the self assessment (using a similar interface to that implemented in TD-5562: Prompt users to ask which features of the framework they want to copy to the assessment

this is done

@rshrirohit
Copy link
Collaborator

@sherif-olaboye, this PR is targeting the Dev branch. I think it should be targeting the Lisbon branch. Could you please check and correct this?

@sherif-olaboye sherif-olaboye changed the base branch from DEV to DLS-Release-v1.3.0 October 2, 2025 11:25
@sherif-olaboye
Copy link
Contributor Author

@sherif-olaboye, this PR is targeting the Dev branch. I think it should be targeting the Lisbon branch. Could you please check and correct this?

Done

Copy link
Collaborator

@rshrirohit rshrirohit left a comment

Choose a reason for hiding this comment

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

  1. Can we please have some sort of division between the list of features to be copied and I agree checkbox? Right now it feels that the I agree checkbox is the continuation of the list
image
  1. If I don't agree to this, the error shows on all list items. Can we please correct this?
image

@sherif-olaboye
Copy link
Contributor Author

  1. Can we please have some sort of division between the list of features to be copied and I agree checkbox? Right now it feels that the I agree checkbox is the continuation of the list
image 2. If I don't agree to this, the error shows on all list items. Can we please correct this? image

Done

Copy link
Collaborator

@rshrirohit rshrirohit left a comment

Choose a reason for hiding this comment

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

Could you please address the inline comments?

@rshrirohit rshrirohit merged commit ca0b53d into DLS-Release-v1.3.0 Oct 30, 2025
3 checks passed
@rshrirohit rshrirohit deleted the Develop/Features/TD-5563-Allowuserstochangetheprimaryframework branch October 30, 2025 14:54
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.

4 participants