Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

#4023 - Additional transportation and bc residency uploaders is mandatory #4278

Conversation

lewischen-aot
Copy link
Collaborator

@lewischen-aot lewischen-aot commented Jan 22, 2025

  • Made the additional transportation exception document uploader field mandatory field for all options for the following areas:
    • Application
    • Change Request
  • Made the BC residency exception document uploader field mandatory field for all options for the following areas:
    • Application

Screenshot for the BC residency file uploader
image

Screenshot for the additional transportation file uploader
image

@lewischen-aot lewischen-aot added Enhancement New feature or request User Story Form.io Form IO definitions changed. labels Jan 22, 2025
@lewischen-aot lewischen-aot self-assigned this Jan 22, 2025
@lewischen-aot lewischen-aot marked this pull request as ready for review January 22, 2025 20:51
@@ -480,7 +479,7 @@
"scrollToTop": false,
"collapsible": false,
"key": "additionTransportationPanel",
"customConditional": "",
"customConditional": "show = (data.transportationCostSituation === 'noLimit' || data.transportationCostSituation === 'special' || data.transportationCostSituation === 'educationPlacement');",
Copy link
Collaborator

Choose a reason for hiding this comment

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

The uploader must be present and required independently of the select option, is it right?
Do we need this condition?

Copy link
Collaborator Author

@lewischen-aot lewischen-aot Jan 23, 2025

Choose a reason for hiding this comment

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

This is added to be consistent with the behavior in the additional transportation section in the student application. I can remove it if this is not needed.

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Custom condition is removed.

Copy link
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.

Nice work, one minor comment only.

Copy link

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 22.52% ( 3864 / 17159 )
Methods: 10.32% ( 224 / 2170 )
Lines: 25.93% ( 3336 / 12866 )
Branches: 14.32% ( 304 / 2123 )

Copy link

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 65.59% ( 589 / 898 )
Methods: 59.63% ( 65 / 109 )
Lines: 68.72% ( 468 / 681 )
Branches: 51.85% ( 56 / 108 )

Copy link

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 86.14% ( 1249 / 1450 )
Methods: 82.42% ( 136 / 165 )
Lines: 88.51% ( 1032 / 1166 )
Branches: 68.07% ( 81 / 119 )

Copy link

E2E SIMS API Coverage Report

Totals Coverage
Statements: 68.11% ( 6019 / 8837 )
Methods: 65.84% ( 742 / 1127 )
Lines: 71.94% ( 4712 / 6550 )
Branches: 48.71% ( 565 / 1160 )

Copy link
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.

Thanks for making the changes, looks good 👍

Copy link
Collaborator

@guru-aot guru-aot left a comment

Choose a reason for hiding this comment

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

LGTM, nice work @lewischen-aot

@lewischen-aot lewischen-aot added this pull request to the merge queue Jan 23, 2025
Merged via the queue into main with commit 3779f84 Jan 23, 2025
21 checks passed
@lewischen-aot lewischen-aot deleted the feature/#4023-additional-transportation-bc-residency-uploader branch January 23, 2025 17:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Enhancement New feature or request Form.io Form IO definitions changed. User Story
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants