-
Notifications
You must be signed in to change notification settings - Fork 5
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
[WIP] Add workflow import command #134
base: master
Are you sure you want to change the base?
Conversation
@hughdickinson Could you look at |
c41610a
to
7f41bac
Compare
Hi Adam. I checked the attributes that were set on the active workflow instance for the Clump Scout project. I got:
I'm not actually sure what most of them refer to and I assume that most of those will be set at some later stage. That said, in addition to what you already have you might want to add (optionally?) |
Yeah, I think we'll want those to be included. They can't be edited in the Python client at the moment, so I'm going to put this on hold for now and include it in the 1.2 release instead (after the Python client is next updated). |
It would be better to use the API to clone a workflows and ensure all sub-resources are copied appropriately, zooniverse/Panoptes-Front-End#4733 (comment) It may be that workflow copying at the API isn't as fully featured as this implementation, however hearing these issues / use case would inform those features being added as we have similar support for whole projects, zooniverse/panoptes#2861 |
I'd be up for adding a copy command too, but I think it also makes sense to allow exporting/importing to/from a local file (the existing |
Closes #73