Skip to content

Conversation

@nkutub
Copy link
Contributor

@nkutub nkutub commented Nov 19, 2025

Ticket

Resolves # TSS-442/

Changes

  • update the certification controller PATCH method to pull the nested params for certification requirements on updates.

Context for reviewers

When creating a new certification under demo, if a user updates the certification, the certification requirments get lost. this ultimately breaks the certification and will not work on other functions in the app. The change can been seen after update on the form, the requirements lose some values .

Testing

When clicking on update the values in the form fields all stay the same, nothing is lost.

image

Preview environment for reporting-app

♻️ Environment destroyed ♻️

@nkutub nkutub requested review from a team as code owners November 19, 2025 19:28
@nkutub nkutub requested a review from giverm November 19, 2025 19:28
@nkutub nkutub force-pushed the TSS-442-fix-update-on-certification-request-page branch from f4c7ad8 to af0152a Compare November 20, 2025 20:57
Co-authored-by: Michael Giver <[email protected]>
@nkutub nkutub merged commit f1aca03 into main Nov 21, 2025
10 of 11 checks passed
@nkutub nkutub deleted the TSS-442-fix-update-on-certification-request-page branch November 21, 2025 21:57
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