Minor improvements in handling offline applications
Summary: Some minor loose ends from exploratory testing: - disconnect device logs event listeners if the device disconnects - show metro if that is still up, even if the app is no longer connected - hide the options in the support form to record videos / screenshots if the device isn't connected Reviewed By: nikoant Differential Revision: D26277100 fbshipit-source-id: 49d0c934d587b226bc25524224efce60b53939e9
This commit is contained in:
committed by
Facebook GitHub Bot
parent
54f9ff260b
commit
3b6be6b7b6
@@ -225,6 +225,7 @@ export default class BaseDevice {
|
||||
}
|
||||
|
||||
disconnect() {
|
||||
this.logListeners.clear();
|
||||
this.connected.set(false);
|
||||
}
|
||||
|
||||
|
||||
@@ -86,6 +86,8 @@ export const PluginList = memo(function PluginList({
|
||||
pluginsChanged,
|
||||
]);
|
||||
const isConnected = useValue(activeDevice?.connected, false);
|
||||
const metroConnected = useValue(metroDevice?.connected, false);
|
||||
const isArchived = activeDevice?.isArchived;
|
||||
|
||||
const annotatedDownloadablePlugins = useMemoize<
|
||||
[
|
||||
@@ -198,7 +200,7 @@ export const PluginList = memo(function PluginList({
|
||||
))}
|
||||
</PluginGroup>
|
||||
|
||||
{isConnected && (
|
||||
{!isArchived && metroConnected && (
|
||||
<PluginGroup
|
||||
key="metro"
|
||||
title="React Native"
|
||||
|
||||
Reference in New Issue
Block a user