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

Fix for walkthrough and show welcome dependency. #931

Merged
merged 1 commit into from
Sep 20, 2024

Conversation

Tatsinnit
Copy link
Member

@Tatsinnit Tatsinnit commented Sep 19, 2024

This fix is purely for the aspect of things which leads to the fixing #930 Now once user select the aka uncheck show welcome page the walk thought will not show up.

because upon uncheck the property in workspace gets set like this, and we are making sure walkthrough is not called until that is set. Thank you so much @ReinierCC for testing and fyi/collab. <3

Screenshot 2024-09-20 at 10 18 07 AM

@Tatsinnit Tatsinnit added bug Something isn't working enhancement 🚀 New feature or request or improvements on existing code. labels Sep 19, 2024
@Tatsinnit Tatsinnit self-assigned this Sep 19, 2024
@Tatsinnit Tatsinnit linked an issue Sep 19, 2024 that may be closed by this pull request
Copy link
Collaborator

@ReinierCC ReinierCC left a comment

Choose a reason for hiding this comment

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

Looks good, tested VSIX as well.

Copy link
Collaborator

@tejhan tejhan left a comment

Choose a reason for hiding this comment

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

Tested & working good! 👍

@Tatsinnit Tatsinnit merged commit c78eb34 into Azure:main Sep 20, 2024
8 checks passed
tejhan pushed a commit to tejhan/vscode-aks-tools that referenced this pull request Dec 4, 2024
Co-authored-by: Tatsat Mishra <tatsat.mishra@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working enhancement 🚀 New feature or request or improvements on existing code.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

AKS Welcome page ALWAYS opens
3 participants