Planner redesign, starred courses and course dialog - #155
Merged
Merged
Conversation
|
Deployment failed for project coursemap with the following error: Learn More: https://vercel.com/coursemap?upgradeToPro=build-rate-limit |
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
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.
Summary
Merge after #154. Replaces #149.
Before: the planner showed every semester at once with a side sheet for course details.
After: the planner has year tabs with progress, the open year's semesters on the left and a scrolling "Courses to plan" box on the right with one tab per degree, major and minor plus Starred. Courses drag into semesters, and holding a drag over a year tab switches year. Courses can be starred and moved into another imported year. Course details open in a dialog with About and Requisites tabs, enrolment steps, cost and availability.
How: three commits (stars and year moves with migrations 019 and 020, course dialog, planner layout). This branch matches
feat/requirement-placementsat 62810d1 apart from Prettier fixes to three files and the test updates in #152, #153 and #154.Verification
tsc, unit, database and browser tests pass; CI green apart from the Vercel preview (daily deploy limit)Deployment notes
Migrations
019_plan_starred_courses.sqland020_plan_item_year_versions.sql. They are applied to haz's local Supabase only. Merging tomainruns the production migration job, which applies 015 to 020 to the live project.