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

Fix disappearing connections after browser refresh in Theia #530 #554

Merged
merged 4 commits into from May 6, 2020

Conversation

ShimonBenYair
Copy link
Contributor

@ShimonBenYair ShimonBenYair commented May 4, 2020

Signed-off-by: Shimon Ben.Yair shimon.ben.yair@sap.com

This PR fixes the problem that after adding a new connection (for example sqlite/hana) it is being saved in the settings.json and is displayed ok, but after refresh it disappears.

Close #530


Thank you for your contribution!
Before submitting this PR, please make sure:

  • Your code builds clean without any errors or warnings
  • You have made the needed changes to the docs
  • You have wrote a description of what is the purpose of this pull request above

@ShimonBenYair
Copy link
Contributor Author

@mtxr
Can you please code review and merge this pull request ?

@ShimonBenYair ShimonBenYair changed the title Fix disappearing connections after refreshing the browser in Theia Fix disappearing connections after browser refresh in Theia #530 May 4, 2020
@mtxr
Copy link
Owner

mtxr commented May 4, 2020

@ShimonBenYair I'll do this tonight. In a hurry here

@mtxr
Copy link
Owner

mtxr commented May 6, 2020

@ShimonBenYair your PR looks fine, although I'ld rather prefer if we could call explorer.updateTreeRoot() in the last line of https://github.com/ShimonBenYair/vscode-sqltools/blob/missingconnections/packages/plugins/connection-manager/extension.ts#L632 instead of making explorer public. This changed in another branch, so if you can move this call to the register method, it should have the same effect and will make easier for me to resolve conflicts in the other branch

How does it sound?

Copy link
Owner

@mtxr mtxr left a comment

Choose a reason for hiding this comment

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

Please check aforementioned comments

@ShimonBenYair
Copy link
Contributor Author

@mtxr

done, please merge

@mtxr mtxr merged commit 640dc89 into mtxr:master May 6, 2020
@ShimonBenYair
Copy link
Contributor Author

@mtxr
Do you know when 0.21.8 version will be released ?

@mtxr
Copy link
Owner

mtxr commented May 6, 2020 via email

@ShimonBenYair
Copy link
Contributor Author

ShimonBenYair commented May 20, 2020

Today EOD

On Wed, May 6, 2020 at 9:10 AM ShimonBenYair @.***> wrote: @mtxr https://github.com/mtxr Do you know when 0.21.8 version will be released ? — You are receiving this because you were mentioned. Reply to this email directly, view it on GitHub <#554 (comment)>, or unsubscribe https://github.com/notifications/unsubscribe-auth/AAFMX2LJH25VO2WCNEKQAPTRQFHR7ANCNFSM4MYTR6YA .

The new version was not released right ?
in the tags i still see 0.21.6
and in the vscode marketplace it is 0.21.7

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

Successfully merging this pull request may close these issues.

Refresh issue with connections saved in settings.json (theia)
2 participants