-
Notifications
You must be signed in to change notification settings - Fork 12
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
Post-Launch Improvements #94
Merged
Conversation
This file contains 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
Revert horizontal bar graph direction change, change GPA graphs to be vertical graphs
Add y-axis formatting, using percents on main grades graph and GPAs with 2 decimal points on others
* Update CardContents.tsx * Update ProfessorCard.tsx
* Shorter AbortController solution * Simpler AbortController Fix
* Student aggregation total count display * Add commas * Remove print statement --------- Co-authored-by: William Skaggs <wskaggs3262@gmail.com>
* MUI Icons * Remove SearchIcon
Percentage taking again sounds like percentage of students that failed and had to retake. Department of <> at University of Texas at Dallas on all professors is redundant.
* Move axis label formatter for horizontal bar graph * Fix type error
clarify language Co-authored-by: William Skaggs <wskaggs3262@gmail.com>
* Add new query format * Implement radix tree to lower file size from 70MB to 24MB Return max of 20 options * Autocomplete refinements * Remove all sections but HON possibility to add HNx and 0Lx sections
* Create main.yml * Add npm install * Shorter npm install and prettier fix * Run prettier --write Update .prettierignore * .prettierignore formatting --------- Co-authored-by: William Skaggs <wskaggs3262@gmail.com>
* Add share button * Fixes Copy correct URL URL copy confirmation White share icon * Remove skip share menu to test copy
* add vercel analytics * update package-lock.json
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
* Cache professor data API calls * Remove professor cards not found * Show box for professor RMP not found * Add cache expiry * Dashboard organization * Remove duplicate professor cards * Remove console.log * Change RMP cache to 1 month * Pad professor not found * Revert ProfessorCard.tsx * Run prettier on failing file --------- Co-authored-by: William Skaggs <wskaggs3262@gmail.com>
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.
Overview
Resolves several bugs and adds quality-of-life improvements, including optimization to reduce autocomplete response time and more robust error handling.
What Changed
Most changes are technical, implementation changes, with a few new features added.