Skip to content
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
wants to merge 48 commits into
base: master
Choose a base branch
from
Open

v1.0.0 #118

wants to merge 48 commits into from

Conversation

aomi
Copy link
Member

@aomi aomi commented Apr 10, 2021

No description provided.

J-Raymer and others added 27 commits February 14, 2021 01:56
* 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
Copy link

codecov bot commented Apr 10, 2021

Codecov Report

Attention: Patch coverage is 92.36364% with 21 lines in your changes missing coverage. Please review.

Project coverage is 86.50%. Comparing base (58389d3) to head (303270e).

Current head 303270e differs from pull request most recent head 7e20405

Please upload reports for the commit 7e20405 to get more accurate results.

Files Patch % Lines
src/common/utils.ts 38.46% 16 Missing ⚠️
src/kuali/catalog.ts 94.64% 3 Missing ⚠️
src/client.ts 98.11% 1 Missing ⚠️
src/dev/path-builders.ts 92.30% 1 Missing ⚠️
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.
📢 Have feedback on the report? Share it here.

keithradford and others added 2 commits May 2, 2021 13:34
* 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
aomi and others added 19 commits May 29, 2021 11:41
* 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
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants