Feature #8689
open
Trigger an arvados job to retrieve the data and import it into Keep
Added by Ward Vandewege almost 9 years ago.
Updated over 8 years ago.
Category:
Third party integration
Description
When the API call described in
#8688 is received, that should trigger
- an email to the participant saying the transfer has been requested/started
- a user log entry to that effect
- an Arvados job that will ingest the data (when that is done, Tapestry needs to get notified or notice; see #8697)
- Description updated (diff)
- Category set to Third party integration
- Assigned To set to Tom Clegg
- Status changed from New to In Progress
Reviewing origin/8697-ruby187-compat:
I see this changes the gemspec dependencies for google-api-client:
- s.add_dependency('google-api-client', '~> 0.6.3', '>= 0.6.3')
+ s.add_dependency('google-api-client', '>= 0.7', '< 0.9')
Does that mean that we no longer need that geriatric google-api-client version for our Ruby sdk?
Yes... we're graduating from a 3-year-old version (0.6.4) to a 1-year-old version (0.8.6).
AFAICT our dependence on 0.6.3 was an accident: According to commit logs, I pinned to 0.6.x to make bundle stop complaining that ">= 0.6.3" was too open-ended (which is kind of silly, given semver doesn't guarantee anything in 0.x).
Now we need <0.9
to avoid a Ruby >= 1.9 dependency. (IIRC one of google-api-client 0.9's dependencies, not google-api-client 0.9 itself, abandoned Ruby 1.8.)
Current version is 0.9.4.
Tom Clegg wrote:
Yes... we're graduating from a 3-year-old version (0.6.4) to a 1-year-old version (0.8.6).
AFAICT our dependence on 0.6.3 was an accident: According to commit logs, I pinned to 0.6.x to make bundle stop complaining that ">= 0.6.3" was too open-ended (which is kind of silly, given semver doesn't guarantee anything in 0.x).
Now we need <0.9
to avoid a Ruby >= 1.9 dependency. (IIRC one of google-api-client 0.9's dependencies, not google-api-client 0.9 itself, abandoned Ruby 1.8.)
Current version is 0.9.4.
Aha. LGTM then, thanks!
reviewing 8689-participant-project @492d441fedc8796a08b078a1d715d265ca7c1ae2: LGTM, thanks!
Reviewing 8689-use-downloaded-filename: LGTM
Also available in: Atom
PDF