Issue 8 more reporting endpoints#10
Merged
dmlb2000 merged 8 commits intopacifica:masterfrom Apr 13, 2017
Merged
Conversation
…e_reporting_endpoints * issue-125-new_endpoints_for_reporting: Cleaning up incorrect behavior in the earliest/latest and transaction search modules Adding in summarization queries for reporting Final tweaks to simplified user query return for Derek This should fix the broken unit testing now More code complexity changes More codeclimate fixes Codeclimate fixes? urllib is picky with 2/3 compatible code library fix for Python 3 compatibility Adding proper testing for earliest/latest Fixes for ‘files.py’ model to allow proper return of files by transaction, cleanup and fixes for earliest_latest date queries Adding in new code to return simplified person record for use in the uploader, as well as testing code Updated testing code to add files_with_tkv search support Adding endpoint for DMS Uploader Formatting and stylistic fixes Updating query returns to provide more detailed info for display purposes Switching out @db_connection_decorator for more robust connect/close behavior using cherrypy request_start/request_end events Changes to better handle searching for partial instrument id values Adding support for getting an instrument list for a user # Conflicts: # metadata/rest/fileinfo_queries/files_with_tkv.py # metadata/rest/instrument_queries/instrument_user_search.py # metadata/rest/proposal_queries/proposal_user_search.py # metadata/rest/reporting_queries/detailed_transactions_list.py # metadata/rest/reporting_queries/query_base.py # metadata/rest/user_queries/query_base.py # metadata/rest/user_queries/user_lookup.py # metadata/rest/user_queries/user_search.py
* master: try except around the db connect (pacifica#9) # Conflicts: # metadata/orm/base.py
…er) like I was supposed to be.
Closed
This file contains hidden or 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
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
3 participants
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.
Description
[Describe what this change achieves]
Issues Resolved
[List any existing issues this PR resolves]
Check List