From f76de99c33f5e469abc58fe03aab0dc4c96606b9 Mon Sep 17 00:00:00 2001 From: jamaljsr <1356600+jamaljsr@users.noreply.github.com> Date: Tue, 17 Mar 2020 05:39:20 -0400 Subject: [PATCH] fix(eclair): minor fixes fo eclair support --- src/__mocks__/electron.js | 3 +++ .../designer/lightning/ConnectTab.tsx | 20 ++++++++++--------- src/components/network/NetworkView.tsx | 9 ++++++++- src/components/terminal/DockerTerminal.tsx | 18 +++++++++++++++++ src/i18n/locales/en-US.json | 2 +- src/lib/docker/dockerService.spec.ts | 2 +- src/lib/docker/dockerService.ts | 4 ++-- 7 files changed, 44 insertions(+), 14 deletions(-) diff --git a/src/__mocks__/electron.js b/src/__mocks__/electron.js index b2aebce59e..d35ed1885f 100644 --- a/src/__mocks__/electron.js +++ b/src/__mocks__/electron.js @@ -7,6 +7,9 @@ module.exports = { dialog: { showSaveDialog: jest.fn(), }, + Menu: { + buildFromTemplate: jest.fn(), + }, process: { env: {}, }, diff --git a/src/components/designer/lightning/ConnectTab.tsx b/src/components/designer/lightning/ConnectTab.tsx index f37a3e6fd3..c4c1d991a5 100644 --- a/src/components/designer/lightning/ConnectTab.tsx +++ b/src/components/designer/lightning/ConnectTab.tsx @@ -175,15 +175,17 @@ const ConnectTab: React.FC = ({ node }) => { size="small" onChange={e => setAuthType(e.target.value)} > - - {l('filePaths')} - - - {l('hexStrings')} - - - {l('base64Strings')} - + {credentials.admin && [ + + {l('filePaths')} + , + + {l('hexStrings')} + , + + {l('base64Strings')} + , + ]} {node.implementation === 'LND' && ( {l('lndConnect')} )} diff --git a/src/components/network/NetworkView.tsx b/src/components/network/NetworkView.tsx index 866f7fc47d..a9a76245fc 100644 --- a/src/components/network/NetworkView.tsx +++ b/src/components/network/NetworkView.tsx @@ -197,7 +197,14 @@ const NetworkView: React.FC> = ({ match }) => { return ( {header} - {showNotice && } + {showNotice && ( + + )} {toggleAsync.error && ( { const { type, name } = useParams(); const termEl = useRef(null); + // add context menu + useEffect(() => { + window.addEventListener( + 'contextmenu', + e => { + e.preventDefault(); + const menu = remote.Menu.buildFromTemplate([ + { role: 'cut' }, + { role: 'copy' }, + { role: 'paste' }, + ]); + menu.popup({ window: remote.getCurrentWindow() }); + }, + false, + ); + }); + useEffect(() => { info('Rendering DockerTerminal component'); document.title = `Polar Terminal | ${name} (${type})`; diff --git a/src/i18n/locales/en-US.json b/src/i18n/locales/en-US.json index baecf30393..e2cc840449 100644 --- a/src/i18n/locales/en-US.json +++ b/src/i18n/locales/en-US.json @@ -20,7 +20,7 @@ "cmps.common.AdvancedOptionsModal.error": "Failed to update options", "cmps.common.CopyIcon.message": "Copied {{label}} to clipboard", "cmps.common.NavMenu.createNetwork": "Create Network", - "cmps.common.NavMenu.manageNodes": "Manage Nodes", + "cmps.common.NavMenu.manageNodes": "Manage Images", "cmps.common.NavMenu.importNetwork": "Import Network", "cmps.common.NavMenu.exportNetwork": "Export Network", "cmps.common.RestartNode.title": "Restart Node", diff --git a/src/lib/docker/dockerService.spec.ts b/src/lib/docker/dockerService.spec.ts index aef484a7f8..ec8974fdab 100644 --- a/src/lib/docker/dockerService.spec.ts +++ b/src/lib/docker/dockerService.spec.ts @@ -443,7 +443,7 @@ describe('DockerService', () => { it('should create volume dirs when the network is started', async () => { composeMock.upAll.mockResolvedValue(mockResult); await dockerService.start(network); - expect(fs.ensureDir).toBeCalledTimes(7); + expect(fsMock.ensureDir).toBeCalledTimes(7); }); it('should call compose.down when a network is stopped', async () => { diff --git a/src/lib/docker/dockerService.ts b/src/lib/docker/dockerService.ts index 1c5b8186d4..d1921e3fef 100644 --- a/src/lib/docker/dockerService.ts +++ b/src/lib/docker/dockerService.ts @@ -115,7 +115,7 @@ class DockerService implements DockerLibrary { */ async start(network: Network) { const { bitcoin, lightning } = network.nodes; - this.ensureDirs(network, [...bitcoin, ...lightning]); + await this.ensureDirs(network, [...bitcoin, ...lightning]); info(`Starting docker containers for ${network.name}`); info(` - path: ${network.path}`); @@ -140,7 +140,7 @@ class DockerService implements DockerLibrary { * @param node the node to start */ async startNode(network: Network, node: CommonNode) { - this.ensureDirs(network, [node]); + await this.ensureDirs(network, [node]); // make sure the docker container is stopped. If it is already started in an error state // then starting it would have no effect await this.stopNode(network, node);