Skip to content
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

cloudant action annotations for dbname erroneously omit bindTime: true #593

Closed
starpit opened this issue Jun 8, 2016 · 3 comments
Closed

Comments

@starpit
Copy link
Contributor

starpit commented Jun 8, 2016

we would like a way to distinguish dbname as a special parameter, one that we suggest the user provide at package binding time, rather than invocation time.

for example, the read-document action throws an exception if dbname is not provided, and it is unlikely that in normal scenarios dbname will be provided as input from a prior action in a sequence, or by a trigger as part of a rule.

whereas the docID field is much more likely to be in-flow data. therefore, dbname should be prompted for at package binding time

Solution: every cloudant action that specifies dbname as required should also specify it as bindTime: true

@starpit starpit changed the title cloudant package annotation for dbName erroneously specifies required: false cloudant package annotation for dbname erroneously specifies required: false Jun 8, 2016
@starpit starpit closed this as completed Jun 8, 2016
@starpit starpit changed the title cloudant package annotation for dbname erroneously specifies required: false cloudant action annotations for dbname erroneously omit bindTime: true Jun 8, 2016
@starpit
Copy link
Contributor Author

starpit commented Jun 8, 2016

All of the cloudant actions that specify dbName as required must also specify bindTime: true

@starpit starpit reopened this Jun 8, 2016
@csantanapr csantanapr added this to the MILESTONE 8 milestone Jun 27, 2016
@sjfink sjfink modified the milestones: MILESTONE 8, MILESTONE 9 Jul 5, 2016
@csantanapr csantanapr added the bug label Aug 5, 2016
@markusthoemmes
Copy link
Contributor

@csantanapr move to catalog?

@markusthoemmes
Copy link
Contributor

Issue moved to openwhisk/openwhisk-package-cloudant #67 via ZenHub

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants