-
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
Release v1.6 #823
Merged
Merged
Release v1.6 #823
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
Person import/export features in one commit
* person import work * first attribute working * both working * condensing code * minor * adding countryadmin back * turning off index signals while importing * working now with shared set of cols * quick fixes * should be working now * new settings; export cols separated * adding index logic * indexing * update * all working now * tests for import and export at requests level * minor fix * remove redundant code * reformatting * reformatting * cleanup * added unit test * starting workadmin * ensuring indexing disabled after person admin testing * ensuring indexing disabled after person admin testing 2 * Cleanup viaf person save test method for new skip lookup config * cleanup and making import model resource extensible for books * book category importing working * book tests passing * book tests passing (minor) * book tests passing (minor 2) * cleanup after review * final cleanups --------- Co-authored-by: rlskoeser <[email protected]>
* Move npm deps needed for production out of dev dependencies * Update npm package versions via npm audit * Include jest types as a non-dev dependency (required for build)
* import work following testing * cleanup * Update mep/common/admin.py Co-authored-by: Rebecca Sutton Koeser <[email protected]> * Update mep/common/admin.py Co-authored-by: Rebecca Sutton Koeser <[email protected]> * cleaning migrations * fixing transaction issues * add categories to edit view * Run import test last to avoid modifying data for other tests --------- Co-authored-by: Rebecca Sutton Koeser <[email protected]> Co-authored-by: rlskoeser <[email protected]>
* minimalist export_creators * export_books modified * export_locations added * Update mep/people/management/commands/export_locations.py Co-authored-by: Rebecca Sutton Koeser <[email protected]> * Update mep/people/management/commands/export_locations.py Co-authored-by: Rebecca Sutton Koeser <[email protected]> * Update mep/people/management/commands/export_locations.py Co-authored-by: Rebecca Sutton Koeser <[email protected]> * cleaning up after tests and review * reverting to default get_object_data logic * fixes to a few errors and loose ends * updated tests to match new export_locations logic * adapting fixtures and tests to plural categories --------- Co-authored-by: Rebecca Sutton Koeser <[email protected]>
Also remove commented out code ref #792
- rename local import/export admin class to distinguish from default - remove unused methods - use bulk operations where possible
Revise import/export updates
Improve viaf death/birth handling
Filter out null values before exporting
Rename address export
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.