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

Adding hidden_value in smart class parameters and variables #543

Merged
merged 1 commit into from
Feb 22, 2016

Conversation

orrabin
Copy link
Member

@orrabin orrabin commented Feb 15, 2016

No description provided.

@@ -39,10 +39,12 @@ Lets configure the *foreman* class to change the user the foreman processes run
* The type of data we want to pass. Most commonly a string, but many other data types are supported. There's no easy way to tell what type of data Puppet is expecting, so you will need to read through the code/documentation that comes with a particular module to find out. Changing the type field requires an appropriately set "Default Value" field.
* Default Value
* This will be imported from Puppet initially, but if Puppet is using any class inheritance, you'll get something unhelpful like "${$foreman::params::user}". This is because Foreman won't follow the inheritance, so you'll need to set a sensible default value
* Hidden value
* Should the values of the smart class parameters be hidden.
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

"..be hidden in the UI."
Also, s/parameters/parameter

@orrabin
Copy link
Member Author

orrabin commented Feb 15, 2016

@tbrisker thanks, fixed your comments.

@domcleal domcleal mentioned this pull request Feb 19, 2016
22 tasks
domcleal added a commit that referenced this pull request Feb 22, 2016
Adding hidden_value in smart class parameters and variables
@domcleal domcleal merged commit 381b69b into theforeman:gh-pages Feb 22, 2016
@domcleal
Copy link
Contributor

Thanks @orrabin!

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

Successfully merging this pull request may close these issues.

None yet

3 participants