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

Insider build connection profile settings are stored in stable user settings file #242

Closed
kburtram opened this issue Oct 31, 2016 · 4 comments
Assignees
Labels

Comments

@kburtram
Copy link
Member

The Insider build's connection profiles settings are stored in the stable build's settings file. We should read\write connection settings from the correct location based on the build type.

This is likely due to our custom code to write into settings file. It would be better to use the new VS Code settings management APIs to avoid this bug.

  1. Create connection profile in Insiders build

Expected: Settings stored in insiders user settings file

Actual: Settings stored in stable user settings file

@kburtram kburtram added the Bug label Oct 31, 2016
@kburtram kburtram changed the title Insider build connection profile settings stored in stable user settings file Insider build connection profile settings are stored in stable user settings file Oct 31, 2016
@MitchellSternke
Copy link
Member

Related: #220 (probably duplicate)

@MitchellSternke
Copy link
Member

Just a note as to why this is blocked: Even though WorkspaceConfiguration.update() allows you to specify whether you want to write to the user or workspace config file, WorkspaceConfiguration.get() does not tell you which config (user or workspace) the read is performed on.

Until this is resolved, there is a potential for data loss if we simply swap out the existing read/write code for handling settings files with the built-in API's since we can't guarantee that we're writing back to the same file that we read from.

@kevcunnane
Copy link
Contributor

It looks like we now have the inspect method available via the API that gives us what we need: {defaultValue: T, globalValue: T, key: string, workspaceValue: T}. See https://code.visualstudio.com/docs/extensionAPI/vscode-api#WorkspaceConfiguration for more info. We should be able to finally resolve in our next update

@llali
Copy link
Member

llali commented Sep 20, 2017

fixed in #991

@llali llali closed this as completed Sep 20, 2017
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

4 participants