check for device type before returning app list

src/commands/installOsuFirmware.js

@@ -2,8 +2,8 @@
 
 import { createCommand, Command } from 'helpers/ipc'
 import { fromPromise } from 'rxjs/observable/fromPromise'
-import { withDevice } from 'helpers/deviceAccess'
 
+import { withDevice } from 'helpers/deviceAccess'
 import installOsuFirmware from 'helpers/firmware/installOsuFirmware'
 
 type Input = {

src/commands/listApps.js

@@ -3,13 +3,16 @@
 import { createCommand, Command } from 'helpers/ipc'
 import { fromPromise } from 'rxjs/observable/fromPromise'
 
+import { withDevice } from 'helpers/deviceAccess'
 import listApps from 'helpers/apps/listApps'
 
-type Input = *
+type Input = {
+  devicePath: string,
+}
 type Result = *
 
-const cmd: Command<Input, Result> = createCommand('manager', 'listApps', () =>
+const cmd: Command<Input, Result> = createCommand('devices', 'listApps', ({ devicePath }) =>
-  fromPromise(listApps()),
+  fromPromise(withDevice(devicePath)(transport => listApps(transport))),
 )
 
 export default cmd

src/components/ManagerPage/AppsList.js

@@ -69,10 +69,17 @@ class AppsList extends PureComponent<Props, State> {
   _unmounted = false
 
   async fetchAppList() {
-    const appsList = CACHED_APPS || (await listApps.send().toPromise())
+    try {
-    CACHED_APPS = appsList
+      const {
-    if (!this._unmounted) {
+        device: { path: devicePath },
-      this.setState({ appsList, status: 'idle' })
+      } = this.props
+      const appsList = CACHED_APPS || (await listApps.send({ devicePath }).toPromise())
+      CACHED_APPS = appsList
+      if (!this._unmounted) {
+        this.setState({ appsList, status: 'idle' })
+      }
+    } catch (err) {
+      this.setState({ status: 'error', error: err.message })
     }
   }
 

src/components/ManagerPage/index.js

@@ -1,8 +1,9 @@
 // @flow
 
-import React, { Component, Fragment } from 'react'
+import React, { Fragment } from 'react'
 import { translate } from 'react-i18next'
 
+import type { Node } from 'react'
 import type { T } from 'types/common'
 
 import AppsList from './AppsList'
@@ -16,44 +17,35 @@ type Props = {
   t: T,
 }
 
-type State = {}
+const ManagerPage = ({ t }: Props): Node => (
-
+  <Fragment>
-class ManagerPage extends Component<Props, State> {
+    <EnsureDevice>
-  render() {
+      {device => (
-    const { t } = this.props
+        <EnsureDashboard device={device}>
-
+          {deviceInfo => (
-    return (
+            <Fragment>
-      <Fragment>
+              {deviceInfo.mcu && <span>bootloader mode</span>}
-        <EnsureDevice>
+              {deviceInfo.final && <span>osu mode</span>}
-          {device => (
+
-            <EnsureDashboard device={device}>
+              {!deviceInfo.mcu &&
-              {deviceInfo => (
+                !deviceInfo.final && (
-                <Fragment>
+                  <EnsureGenuine device={device} t={t}>
-                  {deviceInfo.mcu && <span>bootloader mode</span>}
+                    <FirmwareUpdate
-                  {deviceInfo.final && <span>osu mode</span>}
+                      infos={{
-
+                        targetId: deviceInfo.targetId,
-                  {!deviceInfo.mcu &&
+                        version: deviceInfo.version,
-                    !deviceInfo.final && (
+                      }}
-                      <EnsureGenuine device={device} t={t}>
+                      device={device}
-                        <FirmwareUpdate
+                      t={t}
-                          infos={{
+                    />
-                            targetId: deviceInfo.targetId,
+                    <AppsList device={device} />
-                            version: deviceInfo.version,
+                  </EnsureGenuine>
-                          }}
+                )}
-                          device={device}
+            </Fragment>
-                          t={t}
-                        />
-                        <AppsList device={device} />
-                      </EnsureGenuine>
-                    )}
-                </Fragment>
-              )}
-            </EnsureDashboard>
           )}
-        </EnsureDevice>
+        </EnsureDashboard>
-      </Fragment>
+      )}
-    )
+    </EnsureDevice>
-  }
+  </Fragment>
-}
+)
-
 export default translate()(ManagerPage)

src/helpers/apps/listApps.js

@@ -1,10 +1,23 @@
 // @flow
-
 import axios from 'axios'
+import type Transport from '@ledgerhq/hw-transport'
+
+import { getFirmwareInfo } from 'helpers/common'
+
+const { API_BASE_URL } = process.env
 
-export default async () => {
+export default async (transport: Transport<*>) => {
   try {
+    const { targetId } = await getFirmwareInfo(transport)
+    const { data: deviceData } = await axios.get(
+      `${API_BASE_URL}/device_versions_target_id/${targetId}`,
+    )
     const { data } = await axios.get('https://api.ledgerwallet.com/update/applications')
+
+    if (deviceData.name in data) {
+      return data[deviceData.name]
+    }
+
     return data['nanos-1.4']
   } catch (err) {
     const error = Error(err.message)

src/internals/devices/index.js

@@ -15,6 +15,7 @@ import uninstallApp from 'commands/uninstallApp'
 import installOsuFirmware from 'commands/installOsuFirmware'
 import installFinalFirmware from 'commands/installFinalFirmware'
 import installMcu from 'commands/installMcu'
+import listApps from 'commands/listApps'
 
 export const commands: Array<Command<any, any>> = [
   getAddress,
@@ -31,4 +32,5 @@ export const commands: Array<Command<any, any>> = [
   installOsuFirmware,
   installFinalFirmware,
   installMcu,
+  listApps,
 ]

src/internals/manager/index.js

@@ -1,7 +1,6 @@
 // @flow
 import type { Command } from 'helpers/ipc'
 
-import listApps from 'commands/listApps'
 import getMemInfo from 'commands/getMemInfo'
 
 /**
@@ -20,4 +19,4 @@ import getMemInfo from 'commands/getMemInfo'
  *
  */
 
-export const commands: Array<Command<any, any>> = [listApps, getMemInfo]
+export const commands: Array<Command<any, any>> = [getMemInfo]