Repository navigation
feat(ui): register a Tapis application from the model form - #369
Merged
Merged
Conversation
Add a dialog to the component location section that creates a Tapis application from a pasted or uploaded app.json and makes it public. The browser calls Tapis directly with the signed-in user's token; the model catalog API is not involved. The picker now lists and reads apps from Tapis directly, with listType=ALL and limit=-1 so public apps appear.
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
This branch was successfully deployed
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
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.
Adds a Register a new application dialog to the model form's component location section. The author pastes or uploads an app.json, reviews what it will create, and confirms. DYNAMO creates the app in Tapis, makes it public, selects it, and offers to fill the form's inputs and parameters from it.
Design: docs/design/2026-10-29-dynamo-tapis-app-registration.md, reviewed by security and architecture reviewers.
Tests: 13 dialog tests, library tests for every check and request, existing section tests moved to the direct URLs. Full suite 131 files / 1251 tests, tsc and eslint clean. Removing the share step on purpose fails exactly the two share tests.
Not done here: the guide update (waits on the cookbook guide PR #368), a dialog screenshot (needs the deployed dev site), and a live create on the dev tenant (needs approval at run time). The model catalog API's two Tapis routes still build the Tapis URL from an unchecked tenant; that is a separate issue.