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

Ability to use settings sync when one of the account extensions is disabled #103238

Closed
sandy081 opened this issue Jul 24, 2020 · 6 comments
Closed
Assignees
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders settings-sync verification-needed Verification of issue is requested verified Verification succeeded
Milestone

Comments

@sandy081
Copy link
Member

As a user I would like to disable one of the Microsoft Account or Github Authentication extensions and I can still use Settings Sync.

@sandy081 sandy081 added feature-request Request for new features or functionality settings-sync labels Jul 24, 2020
@sandy081 sandy081 added this to the July 2020 milestone Jul 24, 2020
@sandy081 sandy081 self-assigned this Jul 24, 2020
@sandy081 sandy081 added the verification-needed Verification of issue is requested label Aug 3, 2020
@sandy081
Copy link
Member Author

sandy081 commented Aug 3, 2020

To verify, before turning on settings sync

  • disable any of the auth (Microsoft/Github) account extensions
  • Make sure Turn on Settings Sync action is still available and you can turn on sync

Note: There is a separate issue to track disallowing disabling auth extension if settings sync is using it - #103405

@sandy081
Copy link
Member Author

sandy081 commented Aug 4, 2020

Reverting this feature because of

#103675
#103501

Proper fix needs Settings Sync feature to know until when to wait for the required auth providers are registered which depends on - #103507

@sandy081 sandy081 reopened this Aug 4, 2020
@sandy081 sandy081 removed insiders-released Patch has been released in VS Code Insiders verification-needed Verification of issue is requested labels Aug 4, 2020
@sandy081 sandy081 modified the milestones: July 2020, August 2020 Aug 4, 2020
@sandy081 sandy081 added the verification-needed Verification of issue is requested label Sep 1, 2020
@sandy081
Copy link
Member Author

sandy081 commented Sep 1, 2020

To verify, before turning on settings sync

  • disable any of the auth (Microsoft/Github) account extensions
  • Make sure Turn on Settings Sync action is still available and you can turn on sync
  • When both extensions are disabled, Turn on Settings Sync action is not available.

@lszomoru
Copy link
Member

lszomoru commented Sep 2, 2020

@sandy081, I have disabled both auth extensions yet I am still seeing the Turn on Settings Sync in the Gear and Accounts menus as well as in the command palette. Based on the verification steps that does not seem to be expected. Running Insiders:

Version: 1.49.0-insider
Commit: 00754e4
Date: 2020-09-02T05:43:25.834Z
Electron: 9.2.1
Chrome: 83.0.4103.122
Node.js: 12.14.1
V8: 8.3.110.13-electron.0
OS: Darwin x64 20.0.0

@lszomoru lszomoru added the verification-found Issue verification failed label Sep 2, 2020
@sandy081
Copy link
Member Author

sandy081 commented Sep 2, 2020

Good catch !!!. You shall see the action, not showing the action might confuse users. Instead an error is thrown while turning on sync.

@sandy081 sandy081 reopened this Sep 2, 2020
@sandy081
Copy link
Member Author

sandy081 commented Sep 2, 2020

New steps to verify, before turning on settings sync

  • disable any of the auth (Microsoft/Github) account extensions
  • Make sure Turn on Settings Sync action is still available and you can turn on sync
  • When both extensions are disabled, turning on settings sync action fails with an error.

Note: There is a known issue #105955 if you turn on sync after enabling the auth extension.

@sandy081 sandy081 removed the verification-found Issue verification failed label Sep 2, 2020
@lszomoru lszomoru added the verified Verification succeeded label Sep 3, 2020
@github-actions github-actions bot locked and limited conversation to collaborators Oct 17, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
feature-request Request for new features or functionality insiders-released Patch has been released in VS Code Insiders settings-sync verification-needed Verification of issue is requested verified Verification succeeded
Projects
None yet
Development

No branches or pull requests

4 participants
@lszomoru @sandy081 and others