-
Notifications
You must be signed in to change notification settings - Fork 32
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 UX of "CLI Not Installed" #187
Comments
|
@vcheung-stripe @jbyrum-stripe The new "welcome alert" is quite disruptive to the workflow of opening VS Code without the Stripe CLI installed. Here's a screenshot of me opening VS Code, and the first thing I see is the Stripe alert. Notice that I'm the file explorer, and I haven't even activated the Stripe view in the sidebar. This probably happens because we have changed the extension to activate on I think we should re-consider this UX, and instead explore #57 which would provide a much less in-your-face-UX via WelcomeViews. Example docs: https://code.visualstudio.com/api/references/contribution-points#contributes.viewsWelcome |
We've had a user reach out about the aggressiveness of the reminder here: stripe/stripe-php#1118, another point in favor of revisiting this UX. |
…ated actions. This is in response to #187.
Thanks for reporting this! We're not activating on * but we did have some non-stripe specific events on there. I like the idea of pre-loading the extension but the prompts that end up coming up for the user is super annoying, especially if they aren't even doing anything Stripe related at the moment, and will likely just encourage them to uninstall. It also doesn't look like the loading is heavy enough to notice if we don't pre-load it. |
I have uninstalled this extension. I already have Stripe CLI installed by way of Homebrew. The constant popups are madness. |
@gracegoo-stripe @vcheung-stripe More feedback from our user survey: "I have the CLI installed but it keeps saying I dont in VS code. Makes the extension unusable. " "to many alerts i have zero understanding how to use it yet" |
I think we should definitely implement welcome views (#57). In addition, I've noticed on MacOS that the popup has become much more prominent since the Big Sur update. I think that should have already been reason enough to change this UX. |
When the CLI is not installed:
Bonus points:
The text was updated successfully, but these errors were encountered: