fix(web): do not close all popups on successful rpc requests (#6675)
Fixes regression introduced in 250f43ae
This commit is contained in:
parent
5d159e0ee5
commit
6a212e7e8e
|
@ -57,7 +57,10 @@ export class Remote {
|
||||||
callback.call(context, payload, response_argument);
|
callback.call(context, payload, response_argument);
|
||||||
}
|
}
|
||||||
|
|
||||||
this._connection_alert = null;
|
if (this._connection_alert) {
|
||||||
|
this._connection_alert.close();
|
||||||
|
this._connection_alert = null;
|
||||||
|
}
|
||||||
})
|
})
|
||||||
.catch((error) => {
|
.catch((error) => {
|
||||||
if (error.message === Remote._SessionHeader) {
|
if (error.message === Remote._SessionHeader) {
|
||||||
|
@ -74,8 +77,6 @@ export class Remote {
|
||||||
message:
|
message:
|
||||||
'Could not connect to the server. You may need to reload the page to reconnect.',
|
'Could not connect to the server. You may need to reload the page to reconnect.',
|
||||||
});
|
});
|
||||||
})
|
|
||||||
.finally(() => {
|
|
||||||
this._controller.setCurrentPopup(this._connection_alert);
|
this._controller.setCurrentPopup(this._connection_alert);
|
||||||
});
|
});
|
||||||
}
|
}
|
||||||
|
|
Loading…
Reference in New Issue