-
Notifications
You must be signed in to change notification settings - Fork 1
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
Grant Loader Fibi Migration #117
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Tests pass for me locally. The changes for fibi support look good. The normalization of date handling makes sense.
7bb8a7c
to
6748bb9
Compare
6e0fb87
to
e1a51d6
Compare
e1a51d6
to
3e88f95
Compare
|
3e88f95
to
7d0ccbd
Compare
|
Because it is in the view sql
And add null test to ingest validation
6edd9a8
to
ced30ea
Compare
This PR updates the JHU grant loader connector to use the new JHU grant database Fibi instead of COEUS.
Fibi is a MySQL database, so there were some dependency changes. I also added h2 for testing so non-mock tests could be added for the grant connector class.
Note the
awardEndDate
CLI option date format was changed frommm/dd/yyyy
toyyyy-mm-dd
to be more consistent with thestartDateTime
CLI option format.