-
Notifications
You must be signed in to change notification settings - Fork 535
Add API to get tool launch url for SPA #11760
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
Merged
ofahimIQSS
merged 32 commits into
IQSS:develop
from
GlobalDataverseCommunityConsortium:SPA-_api_to_getToolLaunchUrl
Sep 10, 2025
Merged
Changes from all commits
Commits
Show all changes
32 commits
Select commit
Hold shift + click to select a range
adb81c2
dataset api call
qqmyers 0e97a19
file-level call
qqmyers 9d9d45d
add retention period check to match apis
qqmyers 73243fe
docs
qqmyers fc6e66e
IT tests of dataset and file apis
qqmyers 1a52520
release note
qqmyers 484e47e
fix title underline
qqmyers a9b9f35
missing blank line
qqmyers d3474a4
cleanup error responses per ai review
qqmyers f49527b
use this file as a test
qqmyers 713fe81
handle no params
qqmyers b2788de
Fix returns
qqmyers 54dabdc
don't assume ids are first
qqmyers 0a076aa
only callback on toolurl
qqmyers 705fdab
update apitoken logic
qqmyers 237c7c8
use displayName
qqmyers 1514949
delete /api/admin/test but add some TODOs #11760
pdurbin 520b727
Merge remote-tracking branch 'IQSS/develop' into SPA-_api_to_getToolL…
qqmyers 01c7140
add meetsReqs
qqmyers 65231d6
doc changes per review
qqmyers e8f8aa7
Apply suggestions from code review
qqmyers e16291c
Merge branch 'SPA-_api_to_getToolLaunchUrl' of https://github.com/Glo…
qqmyers 2f9940f
add requirements to etools apis and fix json formatting
qqmyers 4c10b8a
Merge pull request #44 from IQSS/11760-rm-api-admin-test
qqmyers 4b777fb
Merge branch 'SPA-_api_to_getToolLaunchUrl' of https://github.com/Glo…
qqmyers dbd7f8e
update tests
qqmyers 5735be7
add note
qqmyers c799b66
fix test
qqmyers 653be76
add user who can't see draft dataset
qqmyers 932b014
fix path for error msg
qqmyers 6f8ee1b
typo - need user 2
qqmyers 2b9bab9
typo
pdurbin 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 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
| Original file line number | Diff line number | Diff line change |
|---|---|---|
| @@ -0,0 +1,10 @@ | ||
| New API calls have been added to retrieve the URLs needed to launch external tools on specific datasets and files: | ||
|
|
||
| /api/datasets/$DATASET_ID/externalTool/$TOOL_ID/toolUrl | ||
| and | ||
| /api/files/$FILE_ID/externalTool/$TOOL_ID/toolUrl | ||
|
|
||
| If the dataset/file is not public, the caller must authenticate and have permission to view the dataset/file. In such cases, the generated URL will include a callback token containing a signed URL the tool can use to retrieve all the parameters it is configured for. | ||
|
|
||
| Backward incompatibility: | ||
| The responses from the GET /api/externalTools and /api/externalTools/{id} are now formatted as JSON (previously the toolParameters and allowedApiCalls were JSON serialized as strings) and any confiugured "requirements" are included. |
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
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
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
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
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
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.
Uh oh!
There was an error while loading. Please reload this page.