Skip to content

Commit

Permalink
[#21578] Keycard - Factory reset a Keycard
Browse files Browse the repository at this point in the history
  • Loading branch information
@flexsurfer
flexsurfer committed Jan 14, 2025
1 parent 6ffdaa4 commit 9ef178b
Show file tree
Hide file tree
Showing 8 changed files with 175 additions and 14 deletions.
4 changes: 4 additions & 0 deletions src/status_im/contexts/keycard/effects.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -52,6 +52,10 @@
(fn [args]
(keycard/export-key (keycard.utils/wrap-handlers args))))

(rf/reg-fx :effects.keycard/factory-reset
(fn [args]
(keycard/factory-reset (keycard.utils/wrap-handlers args))))

(rf/reg-fx :effects.keycard/sign
(fn [args]
(-> (keycard/sign args)
Expand Down
19 changes: 15 additions & 4 deletions src/status_im/contexts/keycard/error/view.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -2,6 +2,7 @@
(:require [quo.core :as quo]
[react-native.core :as rn]
[status-im.common.events-helper :as events-helper]
[status-im.contexts.keycard.factory-reset.view :as factory-reset]
[utils.i18n :as i18n]
[utils.re-frame :as rf]))

Expand Down Expand Up @@ -31,7 +32,17 @@
:description-text description}]
[rn/view {:style {:margin-horizontal 20}}
[quo/keycard {:holder-name ""}]
[quo/information-box
{:type :default
:style {:margin-top 20}}
(i18n/label :t/unlock-reset-instructions)]]]))
(when (not= :keycard/error.keycard-blank error)
[:<>
[quo/section-label
{:section (i18n/label :t/what-you-can-do) :container-style {:padding-vertical 8}}]
[quo/settings-item
{:title (i18n/label :t/factory-reset)
:image :icon
:image-props :i/placeholder
:action :arrow
:description :text
:description-props {:text (i18n/label :t/remove-keycard-content)}
:on-press (fn []
(rf/dispatch [:show-bottom-sheet
{:content factory-reset/sheet}]))}]])]]))
4 changes: 4 additions & 0 deletions src/status_im/contexts/keycard/events.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -68,6 +68,10 @@
(fn [_ [data]]
{:effects.keycard/export-key data}))

(rf/reg-event-fx :keycard/factory-reset
(fn [_ [data]]
{:effects.keycard/factory-reset data}))

(rf/reg-event-fx :keycard/connect-derive-address-and-add-account
(fn [_ [{:keys [pin derivation-path key-uid account-preferences]}]]
{:fx [[:dispatch
Expand Down
91 changes: 91 additions & 0 deletions src/status_im/contexts/keycard/factory_reset/view.cljs
View file
Original file line number Diff line number Diff line change
@@ -0,0 +1,91 @@
(ns status-im.contexts.keycard.factory-reset.view
(:require [quo.core :as quo]
[react-native.core :as rn]
[status-im.common.events-helper :as events-helper]
[utils.i18n :as i18n]
[utils.re-frame :as rf]))

(defn- reset-card
[]
(rf/dispatch
[:keycard/factory-reset
{:on-success (fn []
(rf/dispatch [:navigate-back])
(rf/dispatch [:keycard/disconnect])
(rf/dispatch [:open-modal :screen/keycard.factory-reset.success]))
:on-failure (fn []
(rf/dispatch [:navigate-back])
(rf/dispatch [:keycard/disconnect])
(rf/dispatch [:open-modal :screen/keycard.factory-reset.fail]))}]))

(defn- connect-and-reset
[key-uid]
(rf/dispatch
[:keycard/connect
{:key-uid key-uid
:on-success reset-card
:on-error (fn [error]
(if (or (= error :keycard/error.keycard-wrong-profile)
(= error :keycard/error.keycard-blank))
(do
(rf/dispatch [:navigate-back])
(rf/dispatch [:keycard/on-application-info-error error]))
(reset-card)))}]))

(defn success-view
[]
[:<>
[quo/page-nav
{:icon-name :i/close
:on-press events-helper/navigate-back}]
[quo/page-top
{:title (i18n/label :t/keycard-reset-success)
:description :text
:description-text (i18n/label :t/keycard-empty-ready)}]
[rn/view {:style {:flex 1}}]
[rn/view {:padding-horizontal 20}
[quo/button {:on-press events-helper/navigate-back}
(i18n/label :t/done)]]])

(defn failed-view
[]
[:<>
[quo/page-nav
{:icon-name :i/close
:on-press events-helper/navigate-back}]
[quo/page-top
{:title (i18n/label :t/keycard-reset-failed)}]
[rn/view {:style {:flex 1}}]
[rn/view {:padding-horizontal 20}
[quo/button {:on-press events-helper/navigate-back}
(i18n/label :t/try-again)]]])

(defn sheet
[]
(let [customization-color (rf/sub [:profile/customization-color])
key-uid (rf/sub [:keycard/key-uid])
[checked? set-checked] (rn/use-state false)]
[:<>
[quo/drawer-top {:title (i18n/label :t/factory-reset-keycard)}]
[quo/text
{:style {:padding-horizontal 20
:padding-top 8
:padding-bottom 8}}
(i18n/label :t/factory-reset-warning)]
[quo/disclaimer
{:checked? checked?
:container-style {:margin-horizontal 20}
:on-change #(set-checked (not checked?))}
(i18n/label :t/key-pair-erased)]
[quo/bottom-actions
{:actions :two-actions
:button-one-label (i18n/label :t/continue)
:button-one-props {:disabled? (not checked?)
:customization-color customization-color
:on-press (fn []
(rf/dispatch [:hide-bottom-sheet])
(connect-and-reset key-uid))}
:button-two-label (i18n/label :t/cancel)
:button-two-props {:type :grey
:on-press (fn []
(rf/dispatch [:hide-bottom-sheet]))}}]]))
41 changes: 32 additions & 9 deletions src/status_im/contexts/settings/keycard/view.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -30,19 +30,42 @@
:style style/keycard-owner-name}
profile-name]]]])

(defn- check-keycard
[]
(rf/dispatch
[:keycard/connect
{:on-error
(fn [error]
(if (= error :keycard/error.keycard-wrong-profile)
(rf/dispatch [:keycard/on-application-info-error error])
(rf/dispatch [:keycard/on-application-info-error error])))}]))

(defn registered-keycards
[]
(let [keycards (rf/sub [:keycard/registered-keycards])]
[:<>
[quo/divider-label
{:counter? false
:tight? true
:blur? true}
(i18n/label :t/registered-keycards)]
[rn/view {:style style/registered-keycards-container}
(for [keycard keycards]
^{:key (:keycard-uid keycard)}
[registered-keycard keycard])]]))
[rn/view {:style {:flex 1}}
[quo/divider-label
{:counter? false
:tight? true
:blur? true}
(i18n/label :t/registered-keycards)]
[rn/view {:style style/registered-keycards-container}
(for [keycard keycards]
^{:key (:keycard-uid keycard)}
[registered-keycard keycard])]]
[quo/text
{:size :heading-2
:weight :semi-bold
:style {:margin-left 20}}
(i18n/label :t/scan-keycard-actions)]
[quo/divider-label (i18n/label :t/tips-scan-keycard)]
[quo/markdown-list {:description (i18n/label :t/remove-phone-case)}]
[quo/markdown-list {:description (i18n/label :t/keep-card-steady)}]
[quo/bottom-actions
{:actions :one-action
:button-one-label (i18n/label :t/ready-to-scan)
:button-one-props {:on-press check-keycard}}]]))

(defn view
[]
Expand Down
15 changes: 14 additions & 1 deletion src/status_im/navigation/screens.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -35,6 +35,7 @@
[status-im.contexts.keycard.create.view :as keycard.create]
[status-im.contexts.keycard.empty.view :as keycard.empty]
[status-im.contexts.keycard.error.view :as keycard.error]
[status-im.contexts.keycard.factory-reset.view :as keycard.factory-reset]
[status-im.contexts.keycard.migrate.fail.view :as keycard.migrate.fail]
[status-im.contexts.keycard.migrate.profile-keys.view :as keycard.migrate.profile-keys]
[status-im.contexts.keycard.migrate.re-encrypting.view :as keycard.re-encrypting]
Expand Down Expand Up @@ -964,7 +965,19 @@
{:name :screen/keycard.create.ready-to-add
:metrics {:track? true}
:options {:insets {:top? true :bottom? true}}
:component keycard.create/ready-to-add}])
:component keycard.create/ready-to-add}

{:name :screen/keycard.factory-reset.success
:metrics {:track? true}
:options {:theme :dark
:insets {:top? true :bottom? true}}
:component keycard.factory-reset/success-view}

{:name :screen/keycard.factory-reset.fail
:metrics {:track? true}
:options {:theme :dark
:insets {:top? true :bottom? true}}
:component keycard.factory-reset/failed-view}])

(defn screens
[]
Expand Down
6 changes: 6 additions & 0 deletions src/status_im/subs/keycard.cljs
View file
Original file line number Diff line number Diff line change
Expand Up @@ -82,3 +82,9 @@
:<- [:keycard]
(fn [keycard]
(get-in keycard [:application-info :initialized?])))

(rf/reg-sub
:keycard/key-uid
:<- [:keycard]
(fn [keycard]
(get-in keycard [:application-info :key-uid])))
9 changes: 9 additions & 0 deletions translations/en.json
View file
Original file line number Diff line number Diff line change
Expand Up @@ -1028,6 +1028,9 @@
"export-key": "Export private key",
"external-link": "External link",
"external-storage-denied": "Access to external storage is denied",
"factory-reset": "Factory reset",
"factory-reset-keycard": "Factory reset this Keycard",
"factory-reset-warning": "All data will be erased from the Keycard. Please ensure you have a backup of your recovery phrase before proceeding with the factory reset.",
"failed": "Failed",
"failed-on": "Failed on",
"failed-to-fetch-community": "Failed to fetch community",
Expand Down Expand Up @@ -1346,6 +1349,7 @@
"key-name-error-taken": "Key pair name already in use",
"key-name-error-too-short": "Key pair name must be at least {{count}} characters",
"key-on-device": "Private key is saved on this device",
"key-pair-erased": "Key pair will be erased from Keycard",
"key-pair-imported-successfully": "{{name}} key pair imported successfully",
"key-pair-migrated-successfully": "Profile key pair successfully migrated",
"key-pair-name-updated": "Key pair name updated",
Expand All @@ -1369,6 +1373,7 @@
"keycard-desc": "Own a Keycard? Store your keys on it; you’ll need it for transactions",
"keycard-dont-ask-card": "Don't ask for card to sign in",
"keycard-empty": "Keycard is empty",
"keycard-empty-ready": "Keycard is empty and ready to be used",
"keycard-enter-new-passcode": "Enter new passcode {{step}}/2",
"keycard-error-description": "Connect the card again to continue",
"keycard-error-title": "Connection lost",
Expand Down Expand Up @@ -1427,7 +1432,9 @@
"keycard-redeem-title": "Redeem to",
"keycard-redeem-tx": "Redeem assets",
"keycard-redeem-tx-desc": "Tap the card to sign and receive assets",
"keycard-reset-failed": "Failed to reset Keycard",
"keycard-reset-passcode": "Reset passcode",
"keycard-reset-success": "Keycard has been reset",
"keycard-success-description": "You may remove the card now",
"keycard-success-title": "Success",
"keycard-unauthorized-operation": "You're unauthorized to perform this operation.\n Please tap valid card and try again.",
Expand Down Expand Up @@ -2140,6 +2147,7 @@
"remove-from-contacts-text": "By removing a user from your contact list you do not hide your wallet address from them",
"remove-group": "Remove group",
"remove-key-pair-and-derived-accounts": "Remove key pair and derived accounts",
"remove-keycard-content": "Remove all content from this Keycard",
"remove-network": "Remove network",
"remove-nickname": "Remove nickname",
"remove-nickname-toast": "You have removed {{secondary-name}}'s nickname",
Expand Down Expand Up @@ -2232,6 +2240,7 @@
"scan-an-address-qr-code": "Scan an address QR code",
"scan-key-pairs-qr-code": "Scan key pairs QR code",
"scan-keycard": "Scan Keycard",
"scan-keycard-actions": "Scan Keycard to see available actions",
"scan-or-enter-a-sync-code": "Scan or enter a sync code",
"scan-or-enter-sync-code": "Scan or enter sync code",
"scan-or-enter-sync-code-seen-on-this-device": "Scan or enter sync code seen on this device",
Expand Down

0 comments on commit 9ef178b

Please sign in to comment.