-
Notifications
You must be signed in to change notification settings - Fork 4
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
Refactor CRIS import script to use S3 and log activities #1445
Merged
Merged
Changes from all commits
Commits
Show all changes
52 commits
Select commit
Hold shift + click to select a range
40a6499
some handy tooling
frankhereford 8d84187
rough-in state tracking and s3 integration
frankhereford 8893115
default uploads db
frankhereford 566c670
import botocore
frankhereford b9c690a
new env variable
frankhereford d9c00ca
extra bit of state for the operation
frankhereford 597ce76
new downloading from s3 mechanics
frankhereford 4d35d23
rough in uploading db
frankhereford 530fee1
exclude the directory of PDFs, which exists even if empty
frankhereford ed2f7c3
remove old functions being replaced
frankhereford 61d3e5f
get out the ID so we can update it later if we succeed
frankhereford f499674
mark completion in the sqlite db
frankhereford ee6fdf8
no tracking binary files, even if tiny
frankhereford 3ae67ec
record the schema itself instead
frankhereford 3ed958c
let's not make that mistake again
frankhereford d8653a4
tag-along change - remove version designation from docker compose file
frankhereford ca4887c
add migration to put the table in pg
frankhereford e9f133a
get pg connection instead of sqlite
frankhereford 19965b4
kick this whole block in an indent
frankhereford c6ae9be
don't include note column per conversation
frankhereford 4b62460
convert function to use pg
frankhereford 8007118
port this function to pg as well
frankhereford a043cdb
and one last function being brought over to PG
frankhereford ffa7b12
refactor for psycopg
frankhereford 9ed0e45
move file after attempted
frankhereford 1102d10
more psycopg refactoring
frankhereford dd0e5ca
reinstate deployment environment directories
frankhereford b1244a3
drop this errant column
frankhereford 9148140
Merge branch 'master' into use-s3-for-cris-delivery
frankhereford 31fda55
don't need this anymore
frankhereford 9041064
don't need this anymore
frankhereford 2642593
remove unused global now
frankhereford df60832
throw exception if no work to do
frankhereford abf7f06
seeya, wouldn't want to be ya
frankhereford 2baf51c
rename symbol
frankhereford 3f84a16
remove unneeded import
frankhereford 01ec3ae
check for file download success and that it's a real zip file
frankhereford 0523abd
really use the log table as a log
frankhereford cffe3c0
update sql to new table columns
frankhereford 7407767
woops, shouldn't have committed something so temporary
frankhereford 3d17545
quell noise
frankhereford ea77275
how has this not been a thing?
frankhereford cdffadf
fixme notation
frankhereford c48a934
destructure the tuple properly
frankhereford c0a3e64
move the extract around
frankhereford 2a7e233
fix moving after you're done
frankhereford 07082c6
remove this old code
frankhereford 1c65ffe
pass additional required info in main()
frankhereford b9ce5eb
outdent db handle handling
frankhereford e2ca36e
add some column comments to the migration
frankhereford 92f2161
tag-along field name correction, h/t @johnclary
frankhereford 925e97b
bump patch version for JS apps
frankhereford File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
Oops, something went wrong.
Oops, something went wrong.
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.
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.
We're using
libmagic
to check that the zips are zips through inspection of the binary file.