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.

Sign up for GitHub

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

[UX][Web][SIMS #1450] UI Content for Institution - Login #1459

Merged
merged 5 commits into from
Oct 17, 2022
Merged

[UX][Web][SIMS #1450] UI Content for Institution - Login #1459

Changes from 4 commits
Commits
Show all changes
5 commits
Select commit Hold shift + click to select a range
23bf2b0
Update institution login content
hellolynn-tbtb Oct 14, 2022
e911cf1
Update insitution login content - test
hellolynn-tbtb Oct 14, 2022
82652bc
Institution content update - test #2
hellolynn-tbtb Oct 14, 2022
f09a05d
Updated error messages
hellolynn-tbtb Oct 14, 2022
46eece7
Comments addressed
hellolynn-tbtb Oct 17, 2022
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
55 changes: 19 additions & 36 deletions sources/packages/web/src/views/institution/Login.vue
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -7,37 +7,25 @@
Welcome to StudentAid BC
</h1>
<p class="mb-5">
Login or sign up here to manage your institution account.
Manage your institution account using your Business or Basic BCeID
User ID. Learn how to setup and access your account
<a
rel="noopener"
target="_blank"
Copy link
Contributor

Choose a reason for hiding this comment

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

can you add class="primary-color" for the brand blue color, i believe else it will be default href color

href="https://studentaidbc.ca/policy-and-procedures/partner-portal-information"
>here</a
>.
</p>
<content-group>
<v-row>
<v-col>
<h3 class="category-header-medium primary-color">Login</h3>
<p class="sign-in-description">
For returning users—login using your BCeID.
</p>
<v-btn
color="primary"
@click="login"
prepend-icon="fa:fa fa-user"
>
Login with BCeID
</v-btn>
</v-col>
<v-col
><h3 class="category-header-medium primary-color">Sign up</h3>
<p class="sign-in-description">
For new users—sign up using your BCeID.
</p>
<v-btn
color="primary"
@click="login"
prepend-icon="fa:fa fa-user-plus"
>
Sign Up with BCeID
</v-btn></v-col
>
</v-row>
<h3 class="category-header-medium primary-color">
Login or register
</h3>
<p class="sign-in-description">
No account? Please register through the BCeID website below.
</p>
<v-btn color="primary" @click="login" prepend-icon="fa:fa fa-user">
Login / Register with BCeID
</v-btn>
</content-group>
</v-col>
<v-col
Expand Down Expand Up @@ -96,9 +84,9 @@ export default {
case props.showBasicBCeIDMessage:
return "No such Business account has been found with BCeID. Please login with your Business BCeId.";
case props.showDisabledUserMessage:
return "Disabled user - you don't have access to the system. Please contact Administrator for more information.";
return "Your access was disabled. Please contact your institution's admin to enable your access.";
case props.showUnknownUserMessage:
return "The user was validated successfully but is not currently allowed to have access to this application. Please contact the Administrator for more information.";
return "Your BCeID User ID is not authorized to access our system yet. If you are using a Basic BCeID User ID, please contact [email protected] for authorization. If you are using a Business BCeID User ID, please contact your institution's admin to add you as a new user. For newly approved Business BCeID accounts, please fully complete the steps to activate your Business BCeID first.";
default:
return false;
}
Expand All @@ -107,8 +95,3 @@ export default {
},
};
</script>
<style scoped>
Copy link
Collaborator

Choose a reason for hiding this comment

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

This was removed but the class name is still consumed in line 23. Is it right?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Thanks @andrewsignori-aot! I will remove the class. Looks like no additional styles are attached, so the class is pointless now.

.sign-in-description {
max-width: 180px;
}
</style>