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

quiet installation of local extensions #26

Merged
merged 2 commits into from Oct 24, 2022

Conversation

akosyakov
Copy link
Member

@akosyakov akosyakov commented Oct 20, 2022

Description

Since many users ignore this notification, only 40% of users who noticed it managed to convert, and around 1% of all users never come back if they try to use we decided to quiet this notification on the connection. Instead it logs and we provide an explicit Install Local Extensions command to force sync at any time which will prompt a user.

Related Issue(s)

Fixes gitpod-io/gitpod#7906

How to test

  • Start debugging an extension from this branch.
  • Disable Setting Sync with Gitpod.
  • Using debug window connect to Gitpod workspace in gitpod.io
  • Check that on connection you don't have notifications, but in logs there are error messages that installing local extension failed with a link to enabling Setting Sync with Gitpod.

Screenshot 2022-10-20 at 14 02 19

  • Click on the remote indicator and select Install Local Extensions new command.

Screenshot 2022-10-20 at 14 05 21

  • This time you should see notifications taking you through the flow.

Screenshot 2022-10-20 at 14 05 26

Release Notes

Add Install Local Extensions command to explicitly install local extensions.
On connection try to install local extensions but quietly without notifications.

Documentation

I will do https://github.com/gitpod-io/website/issues/2933 after this PR is merged and deployed for latest and stable.

@akosyakov akosyakov requested review from mustard-mh and jeanp413 and removed request for mustard-mh October 20, 2022 12:29
Copy link
Contributor

@mustard-mh mustard-mh left a comment

Choose a reason for hiding this comment

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

Code LGTM, going to test it

Copy link
Contributor

@mustard-mh mustard-mh left a comment

Choose a reason for hiding this comment

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

image

Since many users ignore this notification, only 40% of users who noticed it managed to convert, and around 1% of all users never come back if they try to use we decided to quite this notitication on the connection. Instead it logs and we provide an explicit `Install Local Extensions` command to force sync at any time which will prompt a user.
@akosyakov akosyakov force-pushed the ak/quiet_local_extensions_install branch from 8a758b7 to a92cd25 Compare October 24, 2022 10:55
@akosyakov akosyakov merged commit f712d2b into master Oct 24, 2022
@akosyakov akosyakov deleted the ak/quiet_local_extensions_install branch October 24, 2022 11:30
@avelops
Copy link

avelops commented Sep 18, 2023

i have the pre-release ext. turned on, how can i use this?
i'm not familliar with all the steps you wrote in the testing how to

@jeanp413
Copy link
Member

@avelops local extensions are now installed automatically on the remote side. is that not the case for you? If not you can check Gitpod logs to check for any error
image

@avelops
Copy link

avelops commented Sep 18, 2023 via email

@jeanp413
Copy link
Member

Maybe for ssh, but we IDE not

Not sure I'm following, could you be more specific 🤔

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