-
Notifications
You must be signed in to change notification settings - Fork 7
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
v1.0.0 #118
Open
aomi
wants to merge
48
commits into
master
Choose a base branch
from
dev
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
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
* feat: add url builders for courses and course details on kuali * feat: make term an argmuent for most client methods. term is optional will grab the current term if left out * test: update client tests * fix: update course dumper to be term aware * feat: update to include 202105 course info update previous terms too and remove generic courses file * feat: make schedule dumper term aware * feat: add 202105 schedule data * feat: add 202105 section data * test: add 202105 to test suite and disable 202009 * fix: remove comment * refactor: id to catalogId * refactor: move to utils * fix: imports * fix: cleanup * refactor: catalog to catalogId * fix: unskip * refactor: getCatalogForTerm * fix: unskip test * fix: remove scripts outside of coverage/test * test: for getCatalogIdForTerm
* feat: add url builder and test for subjects endpoint * feat: add KualiSubject type * feat: add getSubjects method * feat: dump subjects * refactor: id to catalogId * test: fix
…f dev (#107) * seperate into different workflow and allow running off dev * fix: don't run on PR * fix when each workflow runs * only publish on commit to master Co-authored-by: Isaac Cormack <isaac.m.cormack@gmail.com>
* feat: add camelCase rule to eslint * feat: add ignore for URLs file
…mation Refactor DetailedClassInformation
Timestamp and URL available in the response object
…cies (#116) * chore: update dependencies * fix: eslint config
* refactor: use generic interface for client response use response instead of data use Date type instead of string for datetime * fix: value can be a min/max object * refactor: clean up typings for KualiCatalogItem * fix: not assume the subjec/code input is valid * refactor: add parser function * chore: lint fix * feat: create test helper to get course detail from file * refactor: function name * feat: simple return type * test: for KualiCourseItem
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## master #118 +/- ##
==========================================
+ Coverage 83.85% 86.50% +2.64%
==========================================
Files 10 13 +3
Lines 254 452 +198
Branches 38 89 +51
==========================================
+ Hits 213 391 +178
- Misses 41 61 +20 ☔ View full report in Codecov by Sentry. |
* move to branch off dev * add pre and coreq tests * add more test cases * small change to test case name * refactor data * all -> ALL
* chore: update dependecies * chore: update minor/patch version * chore: update to jest 17 and friends * chore: updates to yargs 17 * fix: remove script
* chore: add static 202109 202201 * test: add 202109 202201 basic tests * feat: add IDs for new terms * test: add some fall 2021 tests * feat: normalize spacing
* Fix: Hours can have a 'or' in them * Updated fix * updated chenges after review * update formatting * updated catalog.ts
* feat: bookstore parser * fix: edge case for multipe sections listed as A01/A02/A03 etc * refactor: parse all courses at once * feat: tests * chore: address comments * chore: add TODO * chore: remove unused HTML
* fix: edge case where no section is listed * refactor: section to be undefined
* feat: add buildings parser * fix: add return type * feat: add tests * chore: update version
… page and Details Course Info page. (#137) * Fix: Hours can have a 'or' in them * Updated fix * updated chenges after review * update formatting * updated catalog.ts * extract course title in detailedClassInformation * fix missing tests for detailClassInformation * fixing failing tests * fix initialization for title
* chore: update cheerio package * fix: imports for cheerio root * chore: update eslint and prettier dev dependencies * fix: remove redundant prettier * chore: execute format * chore: update async * chore: update jest related packages * chore: move to utils * fix: import path * chore: update types and glob * chore: update typescript related * fix: mock path * chore: update dayjs * chore: update jest typings * chore: update yargs and fix breaking api change * chore: npm audit fix * refactor: move client to separate file
* chore: add catalog id * feat: add 202205 calendar files * feat: add basic tests
* feat: add catalog codes * feat: 202209 course data * feat: 202209 schedule data * feat: 202209 section data * feat: 202301 course data * feat: 202301 schedule data * feat: 202301 section data * feat: add 202209 and 202301 to tests * chore: bump to v6
* Add fall term * Update utils.ts
fix 202405 uuid
Add 2024 summer term
add summer 2024 tests
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.
No description provided.