-
Notifications
You must be signed in to change notification settings - Fork 490
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
Dataset files API extension for file display data with pagination and sorting #9693
Merged
Merged
Changes from 38 commits
Commits
Show all changes
45 commits
Select commit
Hold shift + click to select a range
73c3235
Stash: getVersionFiles API extension with pagination and order criter…
GPortas dc31788
Stash: IT tests for getVersionFiles API endpoint WIP
GPortas fff2119
Added: IT tests for getVersionFiles API endpoint
GPortas e8951a4
Removed: unused imports
GPortas 6264fa4
Added: publication date field to data file payload
GPortas 639cff8
Added: getCountGuestbookResponsesByDataFileId API endpoint
GPortas 886a508
Added: canDownloadFile method to FileDownloadServiceBean
GPortas 6ead834
Added: canDataFileBeDownloaded API endpoint
GPortas 9f35bf7
Added: naming refactor and managing not found files in new files API …
GPortas a2bc4d4
Removed: not essential findDataFileOrDie call to avoid extra query
GPortas ec75534
Added: getFileThumbnailClass API endpoint and enhanced test coverage …
GPortas 86865f5
Added: getCountGuestbookResponses PIDs support and param format and d…
GPortas 31c306d
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 489b990
Added: dataTables field to DataFile JSON API payload
GPortas 8c2781b
Added: release notes for 9692
GPortas 867fb8a
Added: endpoint for getting file data tables and missing authenticati…
GPortas f2b374e
Added: new endpoint to the release notes
GPortas 6437e5d
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 2cb77d4
Changed: more realistic test tab file content
GPortas c86a1d7
Refactor: file metadatas query formatting
GPortas 742036a
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 991641b
Refactor: using TypedQuery instead of Query for getFileMetadatas
GPortas 1ff9d90
Refactor: getVersionFiles endpoint invalid orderCriteria error handling
GPortas a8a367a
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 7a9c2b3
Removed: canFileBeDownloaded endpoint and related logic
GPortas b52b33d
Added: getUserPermissionsOnFile endpoint to Access API
GPortas d16264d
Removed: getFileThumbnailClass API endpoint
GPortas 0952a62
Changed: release notes
GPortas 02cecd4
Added: docs for files endpoint pagination and order criteria
GPortas 6722777
Added: docs for file dataTables API endpoint
GPortas 90ef2f1
Added: docs for guestbookResponses/count files API endpoint
GPortas a7b2485
Changed: version files endpoint docs tweak
GPortas cd8e229
wording #9692
pdurbin a64199f
Added: API docs for access file user permissions
GPortas 24944eb
Merge branch '9692-files-api-extension-display-data' of github.com:IQ…
GPortas 719fc67
Changed: guestbookResponses/count endpoint renamed to downloadCount
GPortas 1224311
doc tweaks #9692
pdurbin 0c0ddae
Merge branch 'develop' into 9692-files-api-extension-display-data #9692
pdurbin 4b276db
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 6b59a5b
Merge branch '9692-files-api-extension-display-data' of github.com:IQ…
GPortas 35e4547
Fixed: missing import removed by mistake after develop merge
GPortas 6b45d82
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 0df8ca7
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 6b19648
Merge branch 'develop' of github.com:IQSS/dataverse into 9692-files-a…
GPortas 2e2fb38
Added: getFileDataTables endpoint permission checks for restricted an…
GPortas 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
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,7 @@ | ||
The following API endpoints have been added: | ||
|
||
- /api/files/{id}/downloadCount | ||
- /api/files/{id}/dataTables | ||
- /access/datafile/{id}/userPermissions | ||
|
||
The getVersionFiles endpoint (/api/datasets/{id}/versions/{versionId}/files) has been extended to support pagination and ordering | ||
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
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
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
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
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
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
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.
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.
This addition should be documented in the API guide.
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.
Done.
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.
This one is not done, sorry: 😄
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.
Added now, sorry.
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.
Yes, looks good, thanks!