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

Improve First Use Experience #757

Closed
wants to merge 2 commits into from
Closed

Improve First Use Experience #757

wants to merge 2 commits into from

Conversation

jpogran
Copy link
Contributor

@jpogran jpogran commented Sep 3, 2021

When the extension starts, if there isn't a .terraform folder in the current workspace, this warning will appear:

image

Clicking on Run terraform init opens the command palette with the init command pre-populated.

image

Clicking on More Information will open a browser window to https://www.terraform.io/docs/cli/commands/init.html.

@jpogran jpogran changed the title first use info Improve First Use Experience Sep 3, 2021
@jpogran jpogran self-assigned this Sep 3, 2021
@jpogran jpogran added the enhancement New feature or request label Sep 3, 2021
@jpogran jpogran linked an issue Sep 3, 2021 that may be closed by this pull request
Add a check to see if the current workspace has been initialized. If not, pop a warning message informing the user why that is important for the performance of the extension and provide a set of actions for the user to choose. They can either ignore the message, see documentation, or choose to execute terraform init.

When choosing terraform init, it will open the command pallete and wait for the user to execute. This accomplishes the init task as well as teaching the user how the builtin commands help.
@jpogran
Copy link
Contributor Author

jpogran commented Sep 14, 2021

As discussed in the linked issue, there was a popup message informing the user that terraform init was needed. This proved to be distracting and undesirable, as well as being unable to be quieted/ignored. While this PR does have the ability to set a value to remember not to show the message, there isn't a strong set of use cases where we need to perform this action.

Closing this in favor of other initiatives.

@jpogran jpogran closed this Sep 14, 2021
@radeksimko radeksimko deleted the first-use-info branch September 14, 2021 17:52
@github-actions
Copy link

I'm going to lock this pull request because it has been closed for 30 days ⏳. This helps our maintainers find and focus on the active issues.
If you have found a problem that seems related to this change, please open a new issue and complete the issue template so we can capture all the details necessary to investigate further.

@github-actions github-actions bot locked as resolved and limited conversation to collaborators Oct 15, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Provide quick start information message at startup
1 participant