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

Make extension compatible with other ecosystems by using the generic state API #275

Merged
merged 1 commit into from
Oct 19, 2022

Conversation

xoriath
Copy link
Contributor

@xoriath xoriath commented Oct 18, 2022

…efined before accessing it

This closes #274

window object

VSCode seems to have a fallback/binding of this function on the
window object in the WebView, however THeia does not have this binding.

Theia does however provide the same API as described in
https://code.visualstudio.com/api/extension-guides/webview#persistence

This closes tonybaloney#274
@xoriath xoriath force-pushed the state-from-vscode-and-theia branch from 419aa59 to 12c1541 Compare October 19, 2022 00:40
@codecov-commenter
Copy link

Codecov Report

Merging #275 (12c1541) into master (abdb9fd) will increase coverage by 0.10%.
The diff coverage is 0.00%.

@@            Coverage Diff            @@
##           master    #275      +/-   ##
=========================================
+ Coverage    8.25%   8.35%   +0.10%     
=========================================
  Files           3       3              
  Lines         642     634       -8     
  Branches       99      90       -9     
=========================================
  Hits           53      53              
+ Misses        589     581       -8     
Impacted Files Coverage Δ
src/panel/main.ts 3.27% <0.00%> (ø)
src/panel/pets.ts 7.17% <0.00%> (+0.22%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

@tonybaloney tonybaloney changed the title Get the state API either from VsCode or Theia, and check if it is und… Make extension compatible with other ecosystems by using the generic state API Oct 19, 2022
@tonybaloney tonybaloney merged commit 2a1e61a into tonybaloney:master Oct 19, 2022
@xoriath xoriath deleted the state-from-vscode-and-theia branch October 19, 2022 14:46
tonybaloney added a commit that referenced this pull request Feb 26, 2023
Get the state API either from VsCode or Theia, and check if it is und…
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Extension does not run in a theia environment
3 participants