Skip to content

#5757 - Partner self declare tracker card updates#5850

Merged
weskubo-cgi merged 7 commits intomainfrom
feature/#5757-partner-self-declare-tracker-card
Mar 9, 2026
Merged

#5757 - Partner self declare tracker card updates#5850
weskubo-cgi merged 7 commits intomainfrom
feature/#5757-partner-self-declare-tracker-card

Conversation

@weskubo-cgi
Copy link
Copy Markdown
Collaborator

@weskubo-cgi weskubo-cgi commented Mar 4, 2026

Overview

  • Added support for tracker card for partner self declare for Application and Change Request
  • Updated content for partner declaration for Application and Change Request
  • Unified models and code for partner and parent
  • Bug: Fixed issue with missing fields on supportinguserpartner2026-2027.json because offeringintensity wasn't loaded into the form

Screenshots

Student

Application Status Tracker -> Partner Self Declare (26-27)

image

Application Status Tracker -> Partner Declaration (26-27)

image

Application Status Tracker -> Partner Declaration (25-26)

image

Change Request Status Tracker -> Partner Self Declare (26-27)

image

Change Request Status Tracker -> Partner Declaration (26-27)

image

Change Request Status Tracker -> Partner Self Declare (25-26)

image

Change Request Status Tracker -> Partner Declaration (25-26)

image

Ministry

Application Status Tracker -> Partner Self Declare (26-27)

image

Application Status Tracker -> Partner Declaration (26-27)

image

Application Status Tracker -> Partner Declaration (25-26)

image

E2E Tests

  • sources/packages/backend/apps/api/src/route-controllers/application/_tests_/application.aest.controller.getInProgressApplicationDetails.e2e-spec.ts
image
  • sources\packages\backend\apps\api\src\route-controllers\application\_tests_\application.students.controller.getCompletedApplicationDetails.e2e-spec.ts
image
  • sources/packages/backend/apps/api/src/route-controllers/application/_tests_/application.students.controller.getInProgressApplicationDetails.e2e-spec.ts
image

@weskubo-cgi weskubo-cgi self-assigned this Mar 4, 2026
@weskubo-cgi weskubo-cgi marked this pull request as draft March 4, 2026 20:30
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

This PR updates the application status tracker to support partner self-declare (including partner “unable to report” flows), harmonizes parent/partner supporting-user models, and fixes missing financial fields by ensuring offeringIntensity is loaded and propagated to the supporting-user forms.

Changes:

  • Adds offeringIntensity to supporting-user payloads across API + web to ensure the correct dynamic supporting-user form behavior.
  • Refactors “parent/partner info” models to a shared supporting-user details shape (fullName, status, isAbleToReport), and updates tracker UI logic accordingly.
  • Updates/extends e2e coverage for the modified application in-progress details responses (students/aest) and aligns expected payload shapes.

Reviewed changes

Copilot reviewed 15 out of 15 changed files in this pull request and generated 6 comments.

Show a summary per file
File Description
sources/packages/web/src/types/contracts/supporting-user/SupportingUserContracts.ts Adds offeringIntensity to the supporting-user web contract.
sources/packages/web/src/services/http/dto/Application.dto.ts Harmonizes supporting-user details DTO shape used by the web app.
sources/packages/web/src/components/common/applicationTracker/generic/ApplicationStatusTrackerBanner.vue Provides defaults for optional banner props to avoid undefined values.
sources/packages/web/src/components/common/applicationTracker/InProgress.vue Updates partner/parent tracker banners and adds partner self-declare navigation + action disabling.
sources/packages/web/src/components/common/applicationTracker/ApplicationProgressBar.vue Normalizes prop/event bindings and passes action-allowed state down to InProgress tracker.
sources/packages/web/src/components/common/applicationTracker/ApplicationInProgressChangeRequest.vue Updates change-request tracker UI to use unified supporting-user details and adds partner self-declare behavior.
sources/packages/web/src/components/common/SupportingUserForm.vue Ensures offeringIntensity is included in the supporting-user form model.
sources/packages/backend/apps/api/src/services/supporting-user/supporting-user.service.ts Loads application.offeringIntensity when fetching identifiable supporting users.
sources/packages/backend/apps/api/src/route-controllers/supporting-user/supporting-user.students.controller.ts Returns offeringIntensity from the identifiable supporting-user endpoint.
sources/packages/backend/apps/api/src/route-controllers/supporting-user/models/supporting-user.dto.ts Adds offeringIntensity to the reported supporting-user DTO contract.
sources/packages/backend/apps/api/src/route-controllers/application/models/application.dto.ts Refactors supporting-user details shape returned from application tracker endpoints.
sources/packages/backend/apps/api/src/route-controllers/application/application.controller.service.ts Centralizes mapping of supporting users into the tracker DTO shape.
sources/packages/backend/apps/api/src/route-controllers/application/tests/application.students.controller.getInProgressApplicationDetails.e2e-spec.ts Updates expectations for new supporting-user details response shape.
sources/packages/backend/apps/api/src/route-controllers/application/tests/application.students.controller.getCompletedApplicationDetails.e2e-spec.ts Updates expectations for new supporting-user details response shape in completed + change request scenarios.
sources/packages/backend/apps/api/src/route-controllers/application/tests/application.aest.controller.getInProgressApplicationDetails.e2e-spec.ts Adds/updates AEST coverage for partner waiting/supporting-user details response shape.

@weskubo-cgi weskubo-cgi marked this pull request as ready for review March 6, 2026 19:40

it("Should get application in-progress details when the student has a partner who is able to report but the partner has not entered their supporting data.", async () => {
// Arrange
const student = await saveFakeStudent(db.dataSource);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor, student will be created by the saveFakeApplication if one is not provided.

},
},
);
await db.supportingUser.save([partner]);
Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Minor, no need to be provided as an array.

Copy link
Copy Markdown
Collaborator

@andrewsignori-aot andrewsignori-aot left a comment

Choose a reason for hiding this comment

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

Great work, thanks for the refactor to make parents and partners share the same structures, looks good 👍

@tiago-graf tiago-graf self-requested a review March 7, 2026 02:35
@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 9, 2026

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 20.25% ( 4542 / 22428 )
Methods: 9.61% ( 262 / 2725 )
Lines: 24.54% ( 3894 / 15870 )
Branches: 10.07% ( 386 / 3833 )

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 9, 2026

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 75.23% ( 1066 / 1417 )
Methods: 78.38% ( 116 / 148 )
Lines: 78.68% ( 775 / 985 )
Branches: 61.62% ( 175 / 284 )

@sonarqubecloud
Copy link
Copy Markdown

sonarqubecloud Bot commented Mar 9, 2026

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 9, 2026

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 85.68% ( 1616 / 1886 )
Methods: 85% ( 187 / 220 )
Lines: 88.64% ( 1287 / 1452 )
Branches: 66.36% ( 142 / 214 )

@github-actions
Copy link
Copy Markdown

github-actions Bot commented Mar 9, 2026

E2E SIMS API Coverage Report

Totals Coverage
Statements: 75.73% ( 9414 / 12431 )
Methods: 74.04% ( 1084 / 1464 )
Lines: 80.27% ( 6828 / 8506 )
Branches: 61.03% ( 1502 / 2461 )

@weskubo-cgi weskubo-cgi added this pull request to the merge queue Mar 9, 2026
Merged via the queue into main with commit bca9ec9 Mar 9, 2026
22 checks passed
@weskubo-cgi weskubo-cgi deleted the feature/#5757-partner-self-declare-tracker-card branch March 9, 2026 17:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants