Closes: enso-org/cloud-v2#1325
This PR fixes a bug when we were creating multiple instalces of ProjectManager and didn't dispose it's resources after GC
Demo Presentation
// PUT VIDEO HERE
Context:
Few sentences on the high level context for the change. Link to relevant design docs or discussion.
This Change:
What this change does in the larger context. Specific details to highlight for review:
Make project manager as an external dependency for LocalBackend
Put creating ProjectManager inside ReactQuery
Test Plan:
Go over how you plan to test it. Your test plan should be more thorough the riskier the change is. For major changes, I like to describe how I E2E tested it and will monitor the rollout.
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR:
[ ] The documentation has been updated, if necessary.
[ ] Screenshots/screencasts have been attached, if there are any visual changes. For interactive or animated visual changes, a screencast is preferred.
[ ] All code follows the
Scala,
Java,
TypeScript,
and
Rust
style guides. In case you are using a language not listed above, follow the Rust style guide.
Pull Request Description
Tl;dr
Closes: enso-org/cloud-v2#1325 This PR fixes a bug when we were creating multiple instalces of ProjectManager and didn't dispose it's resources after GC
Demo Presentation
// PUT VIDEO HERE
Context:
Few sentences on the high level context for the change. Link to relevant design docs or discussion.
This Change:
What this change does in the larger context. Specific details to highlight for review:
ProjectManager
insideReactQuery
Test Plan:
Go over how you plan to test it. Your test plan should be more thorough the riskier the change is. For major changes, I like to describe how I E2E tested it and will monitor the rollout.
Important Notes
Checklist
Please ensure that the following checklist has been satisfied before submitting the PR: