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

Degree Templates #624

Open
wants to merge 30 commits into
base: master
Choose a base branch
from
Open

Degree Templates #624

wants to merge 30 commits into from

Conversation

ethanz2003
Copy link

Issue

closes Feature Request — Degree Templates Tab #597

Photos

Before
image

After
image

Sorting by Academic Schools
image

Sorting Alphabetically
image

Additional Info

Code has a similar implementation to the Pathway section. Has yet to include data from interdisciplinary majors such as BCBP and Interdisciplinary Science.

ethanz2003 and others added 21 commits October 23, 2022 20:59
Degree Templates json has information that will be displayed in the Degree Templates section on the Yacs page.
- Added the new tab for templates and will be location to find the reccomended courses for major
- Implementing what will be displayed on the screen but running into a blocker ( cannot print table of all majors onto this new tab)
Translated the code from Pathways.vue which sorts HASS pathways alphabetically to match the format of the degree templates json file.  Also fixed a small typo in the json.
Edited 2 files to help fix the blocker that I ran into the other day.
This will bring us closer to arriving at a screen with all majors of the school.
Next: Will add the templates for each major using the json file with a map of all majors and its given template.
This is documentation for the new planned feature.
It list ideas that we have for the feature and what we have done and what we need to do ect.
Added a bit to the plan text file and fixed some bugs in the code so that it can actually run on the localhost site when compiled.
Each course is now displayed on its own line rather than all on one line.
Found what we need to use to start implementing this feature.
Need to fix blockers and learn more front end code to figure this out.
By next week hopefully will be able to create a table.
Created new file to test if we can mimic the explore code to print out a table
ran scraper to gather schools and their degrees
Copied formatting of Pathways.vue to DegreeTemplates.vue to maintain continuous site structure.
Restructured json file to maintain key names for each school object in dictionary.
Added schools section to the description of the json file. Parsed the file from a python script in personal repo.
updated json with version sorting majors into a list categorized by school
condensed the course text to make printing fit a bit better within the pop-up boxes
Buttons are now displaying all of the degree templates information
changes from personal yacs repo scraper applied to json file in yacs repo
Implemented alphabetical sorting for degrees and some error handling.
DarknessUnder
DarknessUnder previously approved these changes Mar 2, 2023
Copy link

@zhengw5 zhengw5 left a comment

Choose a reason for hiding this comment

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

Everything looks good and works with no bugs, but when you click on Free elective or HASS elective, it brings the user to a page and says course was not found. Clicking on this should either bring us to a page with all possible HASS or Free electives, or it should just not bring us anywhere and act as just some text.

Copy link

@zhangn7 zhangn7 left a comment

Choose a reason for hiding this comment

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

Just a suggestion: You could link the explore page when you click on Science/Math options in the curriculum template. Since you link direct courses link when a specific course is on the curriculum, I think you could link the explore page with the MATH/SCIENCE classes they could take.

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.

5 participants