Skip to content
This repository has been archived by the owner on Apr 18, 2024. It is now read-only.

Commit

Permalink
fix: LSDV-4659: Fix memory leak with LSF destroying
Browse files Browse the repository at this point in the history
  • Loading branch information
Gondragos committed Jul 6, 2023
1 parent 97e26e0 commit 96255d1
Showing 1 changed file with 3 additions and 2 deletions.
5 changes: 3 additions & 2 deletions src/sdk/dm-sdk.js
Original file line number Diff line number Diff line change
Expand Up @@ -161,7 +161,7 @@ export class DataManager {
this.panels = config.panels;
this.spinner = config.spinner;
this.spinnerSize = config.spinnerSize;
this.instruments = prepareInstruments(config.instruments ?? {}),
this.instruments = prepareInstruments(config.instruments ?? {});
this.apiTransform = config.apiTransform ?? {};
this.preload = config.preload ?? {};
this.interfaces = objectToMap({
Expand Down Expand Up @@ -253,7 +253,7 @@ export class DataManager {
}

/**
* @param {impotr("../stores/Action.js").Action} action
* @param {import("../stores/Action.js").Action} action
*/
addAction(action, callback) {
const { id } = action;
Expand Down Expand Up @@ -451,6 +451,7 @@ export class DataManager {
}

destroy(detachCallbacks = true) {
this.destroyLSF();
unmountComponentAtNode(this.root);

if (this.store) {
Expand Down

0 comments on commit 96255d1

Please sign in to comment.