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

Update config var description docs #160

Merged
merged 3 commits into from
Nov 1, 2021
Merged

Update config var description docs #160

merged 3 commits into from
Nov 1, 2021

Conversation

eranelbaz
Copy link
Member

Issue & Steps to Reproduce / Feature Request

Update description

Solution

@eranelbaz eranelbaz requested a review from a team November 1, 2021 13:28
@eranelbaz eranelbaz self-assigned this Nov 1, 2021
@eranelbaz
Copy link
Member Author

1 similar comment
@eranelbaz
Copy link
Member Author

Copy link
Contributor

@omry-hay omry-hay left a comment

Choose a reason for hiding this comment

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

Small comment, also will that include the data resource as well?


### Optional

- **description** (String) a description of the variables
Copy link
Contributor

Choose a reason for hiding this comment

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

Please add that we limit it to 255 chars

Copy link
Member Author

Choose a reason for hiding this comment

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

@@ -34,10 +35,11 @@ resource "env0_configuration_variable" "drop_down" {
### Required

- **name** (String) name to give the configuration variable
- **value** (String) value for the configuration variable
- **value** (String, Sensitive) value for the configuration variable
Copy link
Contributor

Choose a reason for hiding this comment

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

What does it mean? a boolean variable? do you think it's clear enough?

Copy link
Member Author

@eranelbaz eranelbaz Nov 1, 2021

Choose a reason for hiding this comment

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

this is the value of the variable and it is sensitive, meaning it is string but you won't get the value
we just didn't update the docs apparently before

@eranelbaz
Copy link
Member Author

@omry-hay didn't commit that file
now is
66d474e

@liranfarage89 liranfarage89 added ready to merge PR approved - can be merged once the PR owner is ready and removed pending final review labels Nov 1, 2021
@eranelbaz eranelbaz merged commit 81f8649 into main Nov 1, 2021
@eranelbaz eranelbaz deleted the chore-update-docs branch November 1, 2021 13:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
ready to merge PR approved - can be merged once the PR owner is ready
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants