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

Course homepage #90

Open
wants to merge 25 commits into
base: main
Choose a base branch
from
Open

Course homepage #90

wants to merge 25 commits into from

Conversation

colton456p
Copy link
Collaborator

@colton456p colton456p commented Dec 8, 2024

PR Info

I have created a course homepage while trying to keep my code reusable and readable. This course home page shows important information such as the total number of students in a course on the instructors LMS, the total number of students that have accepted Teamable can use their information, the past attributes used, the info on the last team formation, and the completion percentage of the on boarding page. Additionally all the site breadcrumbs are set up to work with the home page and proper page routing has been set up. This code only works with the backend changes made in the Backend routes for homepage + courses page PR.

Code Changes:

  • Courses page has a very simple card UI to select which course you want to enter. The course page is the first page that loads up on login.
  • Homepage shows how much of the onboarding form is complete and helps guides the user to the next step.
  • Homepage shows information on the professors students such as the number of students on their LMS compared to the number of students that have completed the Teamable enrolment quiz.
  • Homepage shows the date of the last team formation as well as the number of attributes used.
  • Homepage shows all of the previously used Team attributes.

@colton456p colton456p requested a review from Opeyem1a December 8, 2024 00:34
@colton456p colton456p changed the title Created course Homepage Course homepage Dec 11, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant