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

#2123 - Content Fixes for Institution Account #2199

Merged
merged 2 commits into from
Aug 17, 2023
Merged

Conversation

ann-aot
Copy link
Contributor

@ann-aot ann-aot commented Aug 16, 2023

  • Updated below contents in the formio and vue file
    image
    image
    image
    image
    image

  • Added a new heading to the destination agreement form and include the subheading and contents in a panel
    image

  • Updated the Section B link in the destination agreement form
    image
    -Did some refactoring in the destination agreement form, as the current heading was not following the figma heading standards

Note:

  1. As discussed in our grooming, I didn't do any update of the 3rd point in the AC, i.e Change all references to "Ministry of Advanced Education" in BC Private designation agreement to "Ministry of Post-Secondary and Future Skills.

New data will always have the new name Ministry of Post-Secondary and Future Skills. Only the old data will have the old name Ministry of Advanced Education

  1. For screenshot 1, i.e AC-1, As per the current logic we won't be able to use TEMP for multiple locations.
    I checked with @michesmith, She confirmed to go with the tooltip content for now and to create a ticket to discuss with the business and the business will possibly come up with a requirement to accommodate it. (created https://app.zenhub.com/workspaces/student-information-management-system-5fce9df5aa1b45000e937014/issues/gh/bcgov/sims/2200)

UPDATE:
Ac-1 context is updated, as per https://app.zenhub.com/workspaces/student-information-management-system-5fce9df5aa1b45000e937014/issues/gh/bcgov/sims/2200

@ann-aot ann-aot self-assigned this Aug 16, 2023
@ann-aot ann-aot added Web Portal Form.io Form IO definitions changed. Content Tasks related to UIUX content labels Aug 16, 2023
Copy link
Collaborator

@sh16011993 sh16011993 left a comment

Choose a reason for hiding this comment

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

LGTM @ann-aot 👍

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 @ann-aot

Copy link
Collaborator

@andrepestana-aot andrepestana-aot left a comment

Choose a reason for hiding this comment

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

Looks good!

Copy link
Collaborator

@dheepak-aot dheepak-aot left a comment

Choose a reason for hiding this comment

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

Looks good 👍

@sonarqubecloud
Copy link

Kudos, SonarCloud Quality Gate passed!    Quality Gate passed

Bug A 0 Bugs
Vulnerability A 0 Vulnerabilities
Security Hotspot A 0 Security Hotspots
Code Smell A 0 Code Smells

No Coverage information No Coverage information
0.0% 0.0% Duplication

@github-actions
Copy link

Backend Unit Tests Coverage Report

Totals Coverage
Statements: 17.87% ( 2141 / 11982 )
Methods: 8.25% ( 126 / 1528 )
Lines: 20.68% ( 1877 / 9077 )
Branches: 10.02% ( 138 / 1377 )

@github-actions
Copy link

E2E Queue Consumers Coverage Report

Totals Coverage
Statements: 72.5% ( 406 / 560 )
Methods: 63.38% ( 45 / 71 )
Lines: 74.53% ( 357 / 479 )
Branches: 40% ( 4 / 10 )

@github-actions
Copy link

E2E Workflow Workers Coverage Report

Totals Coverage
Statements: 49.81% ( 267 / 536 )
Methods: 41.56% ( 32 / 77 )
Lines: 55.33% ( 218 / 394 )
Branches: 26.15% ( 17 / 65 )

@@ -230,7 +230,7 @@
},
{
"label": "Institution location code",
"tooltip": "Unique 4 digit alpha code that is assigned the first time this location is designated in Canada.",
"tooltip": "Unique 4 digit alpha code assigned the first time an institution is designated in Canada. If your institution has not yet been designated, input a random 4-letter code as a temporary Institution Location Code, and contact SABC to revise this code once your permanent code has been received. This code must be a valid federal education institution code prior to students submitting applications",
Copy link
Collaborator

Choose a reason for hiding this comment

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

👍

@github-actions
Copy link

E2E SIMS API Coverage Report

Totals Coverage
Statements: 53.56% ( 3845 / 7179 )
Methods: 50.22% ( 467 / 930 )
Lines: 58.55% ( 3135 / 5354 )
Branches: 27.15% ( 243 / 895 )

@ann-aot ann-aot merged commit 6979c1d into main Aug 17, 2023
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:04 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:05 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:05 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:05 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:05 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:07 — with GitHub Actions Inactive
@ann-aot ann-aot temporarily deployed to DEV August 17, 2023 22:07 — with GitHub Actions Inactive
@ann-aot ann-aot deleted the feature/sims-#2123 branch August 17, 2023 22:30
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Content Tasks related to UIUX content Form.io Form IO definitions changed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants