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

Can we add applyTo to credentials too? #366

Closed
carolynvs opened this issue Apr 29, 2020 · 0 comments · Fixed by #407
Closed

Can we add applyTo to credentials too? #366

carolynvs opened this issue Apr 29, 2020 · 0 comments · Fixed by #407
Projects

Comments

@carolynvs
Copy link
Contributor

Currently, both parameters and outputs support the applyTo field which allows an author to communicate that the paramter/output is only applicable for particular actions. This is helpful especially for required parameters, so that as a user, I don't have to specify a required parameter for actions where it isn't used at all.

At first glance, being able to say "I don't need (rando cloud credentials) to perform the dry-run action" would be very user friendly. It's annoying to have to specify credentials for actions that do not use them just because install needed those credentials.

I would like to understand if applyTo was left off of credentials on purpose or if it was an oversight?

I can see technical / usability challenges around CredentialSets when each action may require a different sets of credentials depending on applyTo. But I am not sure I see any outright problems that would prevent us from adding applyTo to credentials. What do you all think?

@carolynvs carolynvs added this to To do in Roadmap Jan 7, 2021
Roadmap automation moved this from To do to Done Feb 19, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging a pull request may close this issue.

1 participant