#5581 - PTIRU to replace PTIB for Education Programs/Institutions #5682
Conversation
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
added 2 commits
January 26, 2026 13:47
weskubo-cgi
added
Web portal
Form.io
Contributor
There was a problem hiding this comment.
Pull request overview
This pull request renames the regulatory body designation from "PTIB" (Private Training Institutions Branch) to "PTIRU" (Private Training Institutions Regulation Unit) across the system. This is a straightforward rename that affects form definitions and requires data migration for existing records.
Changes:
- Updated three form.io JSON definitions to change the label from "PTIB" to "PTIRU" and the value from "ptib" to "ptiru"
- Created database migrations to update existing Institution and EducationProgram records from "ptib" to "ptiru"
- Added
validateWhenHidden: falseproperty to the regulatory body fields in the forms
Reviewed changes
Copilot reviewed 9 out of 9 changed files in this pull request and generated 4 comments.
Show a summary per file
| File | Description |
|---|---|
| sources/packages/forms/src/form-definitions/institutionprofilecreation.json | Updated regulatory body dropdown to show "PTIRU" instead of "PTIB" and added validateWhenHidden property |
| sources/packages/forms/src/form-definitions/institutionprofile.json | Updated regulatory body dropdown to show "PTIRU" instead of "PTIB" and added validateWhenHidden property |
| sources/packages/forms/src/form-definitions/educationprogram.json | Updated regulatory body dropdown to show "PTIRU" instead of "PTIB", added validateWhenHidden property, and reorganized hideLabel property position |
| sources/packages/backend/apps/db-migrations/src/sql/Institution/Rollback-rename-regulating-body-ptib-to-ptiru.sql | Rollback script to revert regulating_body from 'ptiru' back to 'ptib' in institutions table |
| sources/packages/backend/apps/db-migrations/src/sql/Institution/Rename-regulating-body-ptib-to-ptiru.sql | Migration script to update regulating_body from 'ptib' to 'ptiru' in institutions table |
| sources/packages/backend/apps/db-migrations/src/sql/EducationPrograms/Rollback-rename-regulatory-body-ptib-to-ptiru.sql | Rollback script to revert regulatory_body from 'ptiru' back to 'ptib' in education_programs table |
| sources/packages/backend/apps/db-migrations/src/sql/EducationPrograms/Rename-regulatory-body-ptib-to-ptiru.sql | Migration script to update regulatory_body from 'ptib' to 'ptiru' in education_programs table |
| sources/packages/backend/apps/db-migrations/src/migrations/1769465086815-UpdateEducationProgramRegulatoryBody.ts | TypeScript migration class to execute the education program regulatory body update |
| sources/packages/backend/apps/db-migrations/src/migrations/1769465037191-UpdateInstitutionRegulatingBody.ts | TypeScript migration class to execute the institution regulating body update |
...ages/backend/apps/db-migrations/src/sql/Institution/Rename-regulating-body-ptib-to-ptiru.sql
Outdated
Show resolved
Hide resolved
...end/apps/db-migrations/src/sql/Institution/Rollback-rename-regulating-body-ptib-to-ptiru.sql
Outdated
Show resolved
Hide resolved
...ackend/apps/db-migrations/src/sql/EducationPrograms/Rename-regulatory-body-ptib-to-ptiru.sql
Outdated
Show resolved
Hide resolved
...ps/db-migrations/src/sql/EducationPrograms/Rollback-rename-regulatory-body-ptib-to-ptiru.sql
Outdated
Show resolved
Hide resolved
|
Backend Unit Tests Coverage Report
|
E2E Workflow Workers Coverage Report
|
E2E Queue Consumers Coverage Report
|
weskubo-cgi
changed the title
E2E SIMS API Coverage Report
|
weskubo-cgi
commented
Jan 27, 2026
| "validate": { | ||
| "required": true | ||
| }, | ||
| "validateWhenHidden": false, |
Collaborator
Author
There was a problem hiding this comment.
This was added on save. If there's any risk to this change I can remove. This does seem to be standard in forms.
Collaborator
There was a problem hiding this comment.
No issues. Forms that are not updated frequently may have these changes. It usually reflects the default value of the property.
andrewsignori-aot
approved these changes
Jan 27, 2026
Collaborator
andrewsignori-aot
left a comment
andrewsignori-aot
left a comment
There was a problem hiding this comment.
Nice work, looks good 👍
| "validate": { | ||
| "required": true | ||
| }, | ||
| "validateWhenHidden": false, |
Collaborator
There was a problem hiding this comment.
No issues. Forms that are not updated frequently may have these changes. It usually reflects the default value of the property.
tiago-graf
approved these changes
Jan 28, 2026
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.
Overview
Screenshots
Ministry
View Institution Profile
Edit Institution Profile
Create Institution
Institution
Create/Edit Program
Migration Rollback