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

Plugin Disconnects when interacting with plugin window #702

Merged
merged 2 commits into from
Jul 7, 2023
Merged

Conversation

robertsonbrinker
Copy link
Contributor

https://clearbible.atlassian.net/browse/DI-242

OnLeave runs when the plugin window loses focus. This was causing the plugin to disconnect frequently but we weren't gettting any visual notification of the disconnection because of the problems outlined in DI-188.

Should WebAppProxy?.Dispose(); be run somewhere else if its not going to be run in OnLeave or is it not necessary? perhaps in WindowClosing?

@robertsonbrinker robertsonbrinker requested a review from GeoDirk July 6, 2023 20:45
Copy link
Contributor

@GeoDirk GeoDirk left a comment

Choose a reason for hiding this comment

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

I moved the WebAppProxy?.Dispose(); to the window closing function.

@GeoDirk GeoDirk merged commit a88666e into main Jul 7, 2023
@GeoDirk GeoDirk deleted the DI-242 branch July 7, 2023 14:30
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.

2 participants