-
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
Auto include new courses in autocomplete graph #302
base: develop
Are you sure you want to change the base?
Auto include new courses in autocomplete graph #302
Conversation
…ue-to-outdated-autocomplete-graph
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
Verdict: both change the build script on Vercel to be |
This reverts commit 51ba1f9.
This reverts commit c69b32a.
…ue-to-outdated-autocomplete-graph
I added an action that can be run manually to create a PR that updates develop's autocomplete graph and comobs table from the API |
…ue-to-outdated-autocomplete-graph
…ue-to-outdated-autocomplete-graph
Overview
Resolves #270
An easier way to update search with new courses from the API.
What Changed
Add a new script to fetch the data the autocomplete graph and combo table are based off of.
Following Changes
We could change the build script on Vercel to be
npm run fetchdata && npm run buildautocomplete && npm run buildcombos && npm run build
which would keep production up to date, but not the repo.We could also create a GitHub action to run
npm run fetchdata && npm run buildautocomplete && npm run buildcombos
every month and create a PR intodevelop
ormain
.Not sure which is best...