Skip to content

Commit

Permalink
Fetch by department when selected (#2120)
Browse files Browse the repository at this point in the history
* set state issue

* fix all

* Works, but there seems to be a miss on one of the state variables and pulling items while pulling for the filtered list.

* fmt

* infinite loop of API calls

* Can load all pages

* Collin's changes

Mainly just moving around the setting of the newDepartment variable.

* move to changeselecteddepartment

* we produce enough, but too many

* we produce enough, but too many

* test still fails

* fix dupes

* fix coursescount

* program changes

* fixed duplicate items and deduped code a little. Need to finish refactor of dupe code + make sure callback goes back in when going back to ALL_XXXXX

* Figured out the program issues, stopped duplicate requests and fixed swapping department io on both sides

* prevprops wasn't in componentDidUpdate

* wrong variable name after merge

* wrong variable name after merge

* wrong variable name after merge

* added countAndRetrieveMoreCourses and countAndRetrieveMorePrograms

* behave with far less code except the programs retrieves on every call

* programs fixed again

* hold

* 2/4 fixed

* 3/4 fixed

* remove print statements

* updated for format

* fixed, finally.

* add test for mergeNewObjects

---------

Co-authored-by: Collin Preston <[email protected]>
  • Loading branch information
JenniWhitman and collinpreston authored Mar 18, 2024
1 parent 8905b5b commit 1c67dec
Show file tree
Hide file tree
Showing 4 changed files with 319 additions and 119 deletions.
Loading

0 comments on commit 1c67dec

Please sign in to comment.