#6387 - Content: Additional Transportation Exception - #6506
Draft
weskubo-cgi wants to merge 1 commit into
Draft
Conversation
|
Backend Unit Tests Coverage Report
|
E2E Workflow Workers Coverage Report
|
weskubo-cgi
changed the title
E2E Queue Consumers Coverage Report
|
Contributor
There was a problem hiding this comment.
Pull request overview
Updates additional transportation eligibility and supporting-document content for the 2026–27 full-time and part-time SFAA forms.
Changes:
- Revises transportation eligibility messaging and criteria.
- Updates supporting-document instructions.
- Adjusts hidden-field validation settings.
Reviewed changes
Copilot reviewed 2 out of 2 changed files in this pull request and generated 4 comments.
| File | Findings |
|---|---|
sources/packages/forms/src/form-definitions/sfaa2026-27-pt.json |
Moderate contradiction in transit eligibility guidance at lines 5103 and 5319 (2 votes). Applicant-facing grammar and spacing errors at line 5491 (3 votes). |
sources/packages/forms/src/form-definitions/sfaa2026-27-ft.json |
Moderate contradiction in transit eligibility guidance at lines 6500 and 6714 (3 votes). Applicant-facing grammar and spacing errors at line 6866 (3 votes). |
Suppressed comments (7)
sources/packages/forms/src/form-definitions/sfaa2026-27-ft.json:6714
- The introductory text now advertises additional transportation for a work-integrated learning placement, but this option still only covers a clinical or practicum placement. A student in another work-integrated-learning placement has no matching selection, and the related allowance question also remains limited to clinical/practicum placements. Add and map a supported work-integrated-learning option, or remove it from the introductory text.
"label": "I am in a clinical or practicum placement and the commute to the placement location takes more than two hours roundtrip from your neighborhood by the quickest/more direct route.",
sources/packages/forms/src/form-definitions/sfaa2026-27-ft.json:6591
- The new bullet is grammatically unclear (
takes more than two hours to your school roundtrip from your neighborhood) and changes the established phrase toquickest/more direct. Please use a consistent route order andquickest/most directhere and in the matching radio option.
"content": "<strong> Please confirm that you match the criteria below and are eligible for this exception category.</strong>\n<br />\nYou require additional financial support for transportation due to one of the following criteria:\n<ul>\n<li>There is no public transit, or public transit takes more than two hours to your school roundtrip from your neighborhood by the quickest/more direct route.</li>\n<li>I am in a clinical or practicum placement and the commute to the placement location takes more than two hours roundtrip from your neighborhood by the quickest/more direct route.</li>\n<li>I have special circumstances that require additional travel (work, family responsibilities, living distance from school).</li>\n</ul>",
sources/packages/forms/src/form-definitions/sfaa2026-27-ft.json:6508
- The new radio label is not grammatically complete (
for additional transportation allowance) and switches between the termsallowanceand singularcost. Because this is the Yes/No consent prompt, make the requested outcome explicit and use consistent terminology.
"label": "I have exceptional circumstances that meet the eligibility requirements to be considered for additional transportation allowance and want to be assessed for additional transportation cost.",
sources/packages/forms/src/form-definitions/sfaa2026-27-pt.json:5319
- The introductory text now advertises additional transportation for a work-integrated learning placement, but this option still only covers a clinical or practicum placement. A student in another work-integrated-learning placement has no matching selection, and the related allowance question also remains limited to clinical/practicum placements. Add and map a supported work-integrated-learning option, or remove it from the introductory text.
"label": "I am in a clinical or practicum placement and the commute to the placement location takes more than two hours roundtrip from your neighborhood by the quickest/more direct route.",
sources/packages/forms/src/form-definitions/sfaa2026-27-pt.json:5194
- The new bullet is grammatically unclear (
takes more than two hours to your school roundtrip from your neighborhood) and changes the established phrase toquickest/more direct. Please use a consistent route order andquickest/most directhere and in the matching radio option.
"content": "<strong> Please confirm that you match the criteria below and are eligible for this exception category.</strong>\n<br />\nYou require additional financial support for transportation due to one of the following criteria:\n<ul>\n<li>There is no public transit, or public transit takes more than two hours to your school roundtrip from your neighborhood by the quickest/more direct route.</li>\n<li>I am in a clinical or practicum placement and the commute to the placement location takes more than two hours roundtrip from your neighborhood by the quickest/more direct route.</li>\n<li>I have special circumstances that require additional travel (work, family responsibilities, living distance from school).</li>\n</ul>",
sources/packages/forms/src/form-definitions/sfaa2026-27-pt.json:5103
- This new paragraph is grammatically ambiguous: the first
ifclause is followed byyou must ...and then another comma-separated condition, so the relationship between the transit, commute, placement, and special-circumstance criteria is unclear. Use parallel criteria or an explicit list so applicants can determine eligibility.
"content": "If there is limited or no local transit where you live, you must commute more than one hour each way to and from your post-secondary institution, additional transportation for a work-integrated learning placement is required, or if you have special circumstances that require extra travel, you may be eligible for an additional transportation allowance.",
sources/packages/forms/src/form-definitions/sfaa2026-27-pt.json:5111
- The new radio label is not grammatically complete (
for additional transportation allowance) and switches between the termsallowanceand singularcost. Because this is the Yes/No consent prompt, make the requested outcome explicit and use consistent terminology.
"label": "I have exceptional circumstances that meet the eligibility requirements to be considered for additional transportation allowance and want to be assessed for additional transportation cost.",
💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.
| } | ||
| ], | ||
| "content": "<h4 class=\"category-header-medium-small\">Documents required:</h4>\n<ul>\n<li><b>A statement</b> describing how you calculated your weekly kilometers between your residence and school in relation to the following:</li>\n<ul>\n<li>If it’s required for <b>clinical</b> or <b>practicum purposes</b>, please provide a <b>schedule and letter of explanation which includes the location.</b></li>\n<li>If it’s required for <b>family related responsibilities</b> (e.g. driving children to and from school or daycare), please provide a <b>statement that outlines your circumstances.</b></li>\n<li>If you are <b>unable to move closer to your school,</b> please provide a <b>statement that outlines your circumstances.</b></li>\n</ul>\n</ul>\n<h4 class=\"category-header-medium-small\">AND</h4>\n<ul>\n<li><b>A copy of your vehicle insurance:</b></li>\n<ul>\n<li><b>Must</b> be insured for proper vehicle use (e.g. “to or from work or school a one-way distance in excess of XXkm…”)</li>\n<li><b>Must</b> be listed as a driver of the vehicle or the registered owner.</li>\n</ul>\n</ul>", | ||
| "content": "<h4 class=\"category-header-medium-small\">Documents required:</h4>\n<p>A detailed letter explaining how your circumstances meet one or more of the exception criteria listed above, including an estimate of the weekly travel distance between your home to your institution or practicum location(in kilometers), with a description of the calculation method you used to determine the weekly kilometers traveled.</p>", |
| } | ||
| ], | ||
| "content": "<h4 class=\"category-header-medium-small\">Documents required:</h4>\n<ul>\n<li><b>A statement</b> describing how you calculated your weekly kilometers between your residence and school in relation to the following:</li>\n<ul>\n<li>If it’s required for <b>clinical</b> or <b>practicum purposes</b>, please provide a <b>schedule and letter of explanation which includes the location.</b></li>\n<li>If it’s required for <b>family related responsibilities</b> (e.g. driving children to and from school or daycare), please provide a <b>statement that outlines your circumstances.</b></li>\n<li>If you are <b>unable to move closer to your school,</b> please provide a <b>statement that outlines your circumstances.</b></li>\n</ul>\n</ul>\n<h4 class=\"category-header-medium-small\">AND</h4>\n<ul>\n<li><b>A copy of your vehicle insurance:</b></li>\n<ul>\n<li><b>Must</b> be insured for proper vehicle use (e.g. “to or from work or school a one-way distance in excess of XXkm…”)</li>\n<li><b>Must</b> be listed as a driver of the vehicle or the registered owner.</li>\n</ul>\n</ul>", | ||
| "content": "<h4 class=\"category-header-medium-small\">Documents required:</h4>\n<p>A detailed letter explaining how your circumstances meet one or more of the exception criteria listed above, including an estimate of the weekly travel distance between your home to your institution or practicum location(in kilometers), with a description of the calculation method you used to determine the weekly kilometers traveled.</p>", |
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
Screenshots
Full-time
Part-time