-
Notifications
You must be signed in to change notification settings - Fork 4
Uat rog #77
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
Open
rogilmore
wants to merge
32
commits into
databrary:master
Choose a base branch
from
NYU-Databrary:uat-rog
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Uat rog #77
Conversation
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
… functions, and update documentation. Adjust tests for consistency and add new tests for recent changes.
The Databrary API now returns different field names (institutions, affiliates, investigators, hours_of_recordings) instead of the legacy fields (authorized_users, total_volumes, etc.). Updated the function to map new fields while keeping legacy fields as NA for backwards compatibility.
- Add API constant for funder detail endpoint - Implement get_funder_by_id() function - Add test suite (11 test cases) - Add function documentation
- Add API constant for tag detail endpoint - Implement get_tag_by_id() function - Add test suite (12 test cases) - Add function documentation
- Added API_CATEGORY_DETAIL constant - Implemented get_category_by_id() to retrieve category by ID - Add test suite (12 test cases) - Add function documentation
- Added API_CATEGORIES constant - Implemented list_categories() to retrieve all categories - Add test suite (10 test cases) - Add function documentation
- Added API_VOLUME_RECORDS constant - Implemented list_volume_records() to retrieve participant records from volumes - Add test suite (14 test cases) - Add function documentation
- Added API_VOLUME_RECORD_DETAIL constant - Implemented get_volume_record_by_id() to retrieve single record by ID - Created test suite (14 test cases)
- Added API_VOLUME_COLLABORATOR_DETAIL constant - Implemented get_volume_collaborator_by_id() to retrieve single collaborator by ID - Returns detailed collaborator data with user, sponsor, and sponsorship info - Created test suite (15 test cases)
DAT-1493: Updated OAuth2 authentication process and token management
DAT-1494: Updated and added all non-download functionality and tests
DAT-1495: Added and adjusted functions for downloading
fix/DAT-1494
…ions; Changed some default parameters to make function testing easier.
… minor edits to function documentation; add .progress parameter to some `purrr()` calls; comment-out legacy checks on rq parameter for some functions.
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.
This branch includes many changes to the function documentation so that calls without parameters return public data. The branch also bumps the package version to 1.0.0.9000.