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

replace throw with warning when storage is set without workspace #9137

Merged
merged 1 commit into from
Mar 2, 2021

Conversation

amiramw
Copy link
Member

@amiramw amiramw commented Mar 1, 2021

The functionality is not expected to work according to API, however it doesn't fail in VS Code

Signed-off-by: Amiram Wingarten amiram.wingarten@sap.com

What it does

Fixed #8963

How to test

  1. add SQLTOOLS plugin to Theia.
  2. close Workspace and open a file
  3. close file and no " Cannot save data: no opened workspace" error in terminal.

Review checklist

Reminder for reviewers

@amiramw
Copy link
Member Author

amiramw commented Mar 1, 2021

any idea why the build is not starting for this PR?

@vince-fugnitto
Copy link
Member

any idea why the build is not starting for this PR?

@amiramw you can try performing an amend and force push to re-trigger the build.

The functionality is not expected to work according to API, however it doesn't fail in VS Code

Signed-off-by: Amiram Wingarten <amiram.wingarten@sap.com>
@vince-fugnitto vince-fugnitto added quality issues related to code and application quality vscode issues related to VSCode compatibility labels Mar 1, 2021
Copy link
Member

@vince-fugnitto vince-fugnitto left a comment

Choose a reason for hiding this comment

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

I'm fine with the approach of not throwing the error and instead warning that we cannot save the storage.

@amiramw amiramw merged commit 159f175 into master Mar 2, 2021
@amiramw amiramw deleted the warn branch March 2, 2021 18:57
@github-actions github-actions bot added this to the 1.12.0 milestone Mar 2, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
quality issues related to code and application quality vscode issues related to VSCode compatibility
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Request 'setStorageValue' failed on closing file without workspace when SQLTools extension is install
2 participants