This repository has been archived by the owner on Dec 29, 2023. It is now read-only.
Add Artifact loading support to API and UI #19
Closed
Conversation
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
Adds cancel button to job pages, fixes qiime2#17 Adds smart container around job pages for statelessness Adds an artifacts route to the API Dynamically fetches artifacts and their data from the cwd
Instead of pulling the router from Context, change to using the react-router 2.4.x withRouter helper to pass the router to props instead. Will require a call to npm install on any development environments to update react-router
Adds DELETE api endpoint for artifacts that handles deletion of the files Adds confirmation dialog to UI to prevent accidental deltions. Should be changed to custom confirm dialog through React eventually instead of browser alert confirmation style Adds Access-Control-Allow-Methods to response headers, as DELETE was being denied without it.
Adds input artifact storage by the name, as used in Q2D3. Adds Job page dispatch and state mapping Adds pre-commit npm test run hook (needs npm install ran) Adds a jobs reducer for the next steps Adds a test for the jobs reducer
MAINT: Update to use new framework API
Filter out plugins without workflows
Adds cancel button to job pages, fixes qiime2#17 Adds smart container around job pages for statelessness Adds an artifacts route to the API Dynamically fetches artifacts and their data from the cwd
Instead of pulling the router from Context, change to using the react-router 2.4.x withRouter helper to pass the router to props instead. Will require a call to npm install on any development environments to update react-router
Adds DELETE api endpoint for artifacts that handles deletion of the files Adds confirmation dialog to UI to prevent accidental deltions. Should be changed to custom confirm dialog through React eventually instead of browser alert confirmation style Adds Access-Control-Allow-Methods to response headers, as DELETE was being denied without it.
Adds input artifact storage by the name, as used in Q2D3. Adds Job page dispatch and state mapping Adds pre-commit npm test run hook (needs npm install ran) Adds a jobs reducer for the next steps Adds a test for the jobs reducer
Slight API tweaks created conflicts. Fixed.
Sign up for free
to subscribe to this conversation on GitHub.
Already have an account?
Sign in.
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 support on front-end and backend for loading qtf artifacts from the current working directory.
Updates
react-routerto allow usage of thewithRoutercomponent wrapper. Requiresnpm installon all existing dev environments in order to update the package.TODO: