Fix device type checks based on alias

This PR fixes checks for recovery devices and recovery phrases that were based on the device name. This change is required because the public endpoints no longer give information about the alias.
dfinity · Feb 13, 2023 · dd2b8de · dd2b8de
1 parent f3fd4f8
commit dd2b8de
Show file tree

Hide file tree

Showing 2 changed files with 6 additions and 3 deletions.
diff --git a/src/frontend/src/flows/recovery/chooseRecoveryMechanism.ts b/src/frontend/src/flows/recovery/chooseRecoveryMechanism.ts
@@ -105,6 +105,9 @@ export const chooseRecoveryMechanism = async ({
 };
 
 const hasRecoveryPhrase = (devices: DeviceData[]): boolean =>
-  devices.some((device) => device.alias === "Recovery phrase");
+  devices.some((device) => "seed_phrase" in device.key_type);
 const hasRecoveryKey = (devices: DeviceData[]): boolean =>
-  devices.some((device) => device.alias === "Recovery key");
+  devices.some(
+    (device) =>
+      !("seed_phrase" in device.key_type) && "recovery" in device.purpose
+  );
diff --git a/src/frontend/src/flows/recovery/pickRecoveryDevice.ts b/src/frontend/src/flows/recovery/pickRecoveryDevice.ts
@@ -44,7 +44,7 @@ export const init = (devices: DeviceData[]): Promise<DeviceData> =>
         html`<div class="deviceItemAlias">
           <button class="c-button c-button--secondary">
             <span aria-hidden="true"
-              >${device.alias === "Recovery Phrase"
+              >${"seed_phrase" in device.key_type
                 ? seedPhraseIcon
                 : securityKeyIcon}</span
             >