Repository navigation
Fix ntid classes - #366
Merged
Merged
Fix ntid classes#366
Conversation
BigSpaceships
added a commit
that referenced
this pull request
Oct 26, 2026
* Added banner stating that schedule maker isn't working * Revert "Added banner stating that schedule maker isn't working" This reverts commit 371d9e6. * Merge pull request #366 from ComputerScienceHouse/fix-ntid-classes Fix ntid classes * I updated the dump process to handle 3 new course fields: Prerequisites, Typically Offered, and Contact Hours. * I added the new column to the CREATE TABLE statement. * Updated the insertOrUpdateCourse code. * Fixed syntax errors. * Changing contact hours to float. * I added a new script, but idk if I should have NOT NULL or not. * I added defaults. * I missed this file... * I was able to get ScheduleMaker to build. * Updated scripts and php. * I fixed a bug with parsing. * Tested and added a bit more to my migration script. * Undo changes on existing migrations. * Added back a space. * I added a new migration script. * Removed space. * I added a script to update the buildings. * Fixed typo. * I found another missing value. * Added more missing buildings. * Updated PAC. * Updated a tiny syntax error from the recent migration. * Updated multi query error handling for php 8.4. * Updated docker version in processDump. --------- Co-authored-by: tmcnulty387 <tmcnulty387@gmail.com> Co-authored-by: Mobmaker <45888585+Mobmaker55@users.noreply.github.com> Co-authored-by: Mary <Ipadlover8322@gmail.com> Co-authored-by: Noah Hanford <bigspaceships56@gmail.com>
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.
What
Fixes the class browser for N3DG
Why
We got a kind email about it
Test Plan
It built and I got it
Env Vars
Nopalope
Documentation
Nope
Checklist