status-im
/
status-mobile
mirror of https://github.com/status-im/status-mobile.git
2
0
Fork 0
Code Issues Projects Releases Wiki Activity

feat!: add key pairs and accounts settings to wallet settings (#20464) 

* tweak: use support blur colors for action drawer icons

* tweak: support blur colors for drawer-top icon

* tweak: support blur colors for account list inside keypair component

* tweak: use blur colors for missing-keypair bottom sheet actions

* feat: remove feature-flag for key-pairs and accounts inside wallet settings

* fix: ensure key-pairs-and-accounts settings are visible

* tweak: display key-pair rename screen as full-screen modal

* tweak: animate key-pair scanning screen as modal

* fix: do not allow user to rename key-pair to an existing key-pair name

* fix: ensure we visualise the validation error for key-pair name that is too long

* fix: ensure we can see the recovery-phrase suggestions when testnet-mode is active

* tweak: show recovery-phrase errors when keyboard is hidden

* fix: ensure we do not re-hash password

* fix: ensure we call `on-close` function when successfully handling biometrics auth

* fix: ensure we clear error states when editing the recovery-phrase input

* fix: use blur styles for standard authentication password input

* chore: add labels for qr validation

* tweak: handle display import-qr error and allow for re-scanning

* tweak: update blur background for all bottom-sheets

* chore: add feature-flag for import-all key-pairs button for missing key-pairs

* tidy: remove unused variable

* fix: ensure layout for android is correct when entering seed-phrase in testnet-mode

* fix: ensure we return the updated db when updating the db with backup key-pair

* tweak: coerce nil to empty string

* tweak: change validation error message for short key-pair name

* tweak: hide the options icon for the default key-pair inside key-pairs and accounts settings
This commit is contained in:
Sean Hagstrom 2024-06-26 13:16:00 +01:00 committed by GitHub
parent 6931fd0052
commit 256f3f8f24
No known key found for this signature in database
GPG Key ID: B5690EEEBB952194
30 changed files with 245 additions and 177 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

13
src/quo/components/drawers/action_drawers/view.cljs
View File

@ -8,10 +8,11 @@
[react-native.core :as rn])) [react-native.core :as rn]))
(defn- get-icon-color (defn- get-icon-color
[danger? theme] [blur? danger? theme]
(if danger? (cond
(colors/theme-colors colors/danger-50 colors/danger-60 theme) danger? (colors/theme-colors colors/danger-50 colors/danger-60 theme)
(colors/theme-colors colors/neutral-50 colors/neutral-40 theme))) blur? colors/white-opa-70
:else (colors/theme-colors colors/neutral-50 colors/neutral-40 theme)))
(defn- divider (defn- divider
[theme blur?] [theme blur?]
@ -51,7 +52,7 @@
:accessible true :accessible true
:style (style/left-icon sub-label)} :style (style/left-icon sub-label)}
[icon/icon icon [icon/icon icon
{:color (or icon-color (get-icon-color danger? theme)) {:color (or icon-color (get-icon-color blur? danger? theme))
:no-color no-icon-color? :no-color no-icon-color?
:size 20}]]) :size 20}]])
[rn/view [rn/view
@ -84,7 +85,7 @@
:accessible true :accessible true
:accessibility-label :right-icon-for-action} :accessibility-label :right-icon-for-action}
[icon/icon right-icon [icon/icon right-icon
{:color (get-icon-color danger? theme) {:color (get-icon-color blur? danger? theme)
:size 20}]]) :size 20}]])
(when (= state :selected) (when (= state :selected)
[rn/view {:style style/right-icon} [rn/view {:style style/right-icon}

4
src/quo/components/drawers/drawer_top/view.cljs
View File

@ -18,7 +18,7 @@
(defn- left-image (defn- left-image
[{:keys [type customization-color account-avatar-emoji account-avatar-type icon-avatar [{:keys [type customization-color account-avatar-emoji account-avatar-type icon-avatar
profile-picture]}] profile-picture blur?]}]
(case type (case type
:account [account-avatar/view :account [account-avatar/view
{:customization-color customization-color {:customization-color customization-color
@ -28,6 +28,7 @@
:keypair [icon-avatar/icon-avatar :keypair [icon-avatar/icon-avatar
{:icon icon-avatar {:icon icon-avatar
:border? true :border? true
:blur? blur?
:color :neutral}] :color :neutral}]
:default-keypair [user-avatar/user-avatar :default-keypair [user-avatar/user-avatar
@ -213,6 +214,7 @@
[rn/view {:style style/left-container} [rn/view {:style style/left-container}
[left-image [left-image
{:type type {:type type
:blur? blur?
:customization-color customization-color :customization-color customization-color
:account-avatar-emoji account-avatar-emoji :account-avatar-emoji account-avatar-emoji
:account-avatar-type account-avatar-type :account-avatar-type account-avatar-type

3
src/quo/components/list_items/account_list_card/view.cljs
View File

@ -31,7 +31,8 @@
:size :paragraph-2} :size :paragraph-2}
(:name account-props)] (:name account-props)]
[address-text/view [address-text/view
{:networks networks {:blur? blur?
:networks networks
:address (:address account-props) :address (:address account-props)
:format :short}]]] :format :short}]]]
(when (= action :icon) (when (= action :icon)

5
src/quo/components/wallet/keypair/view.cljs
View File

@ -47,8 +47,9 @@
:accessibility-label :title} :accessibility-label :title}
[text/text {:weight :semi-bold} [text/text {:weight :semi-bold}
(if (= type :default-keypair) (keypair-string full-name) full-name)] (if (= type :default-keypair) (keypair-string full-name) full-name)]
(if (= action :selector) (case action
[selectors/view :none nil
:selector [selectors/view
{:type :radio {:type :radio
:checked? selected? :checked? selected?
:blur? blur? :blur? blur?

5
src/quo/components/wallet/missing_keypairs/view.cljs
View File

@ -11,7 +11,7 @@
[utils.i18n :as i18n])) [utils.i18n :as i18n]))
(defn title-view (defn title-view
[{:keys [keypairs blur? on-import-press]}] [{:keys [keypairs blur? on-import-press show-import-all?]}]
(let [theme (quo.theme/use-theme)] (let [theme (quo.theme/use-theme)]
[rn/view [rn/view
{:accessibility-label :title {:accessibility-label :title
@ -29,12 +29,13 @@
:style {:color colors/warning-60}} :style {:color colors/warning-60}}
(i18n/label :t/amount-missing-keypairs (i18n/label :t/amount-missing-keypairs
{:amount (str (count keypairs))})] {:amount (str (count keypairs))})]
(when show-import-all?
[button/button [button/button
{:type :outline {:type :outline
:background :blur :background :blur
:size 24 :size 24
:on-press on-import-press} :on-press on-import-press}
(i18n/label :t/import)]] (i18n/label :t/import)])]
[text/text [text/text
{:size :paragraph-2 {:size :paragraph-2
:style (style/subtitle blur? theme)} :style (style/subtitle blur? theme)}

10
src/status_im/common/bottom_sheet/style.cljs
View File

@ -1,7 +1,6 @@
(ns status-im.common.bottom-sheet.style (ns status-im.common.bottom-sheet.style
(:require (:require
[quo.foundations.colors :as colors] [quo.foundations.colors :as colors]))
[react-native.platform :as platform]))
(def ^:private sheet-border-radius 20) (def ^:private sheet-border-radius 20)
@ -22,11 +21,8 @@
:left 0 :left 0
:right 0}) :right 0})
(defn shell-bg (def shell-bg
[blur-background] {:background-color colors/bottom-sheet-background-blur
{:background-color (if blur-background
blur-background
(if platform/ios? colors/white-opa-5 colors/neutral-100-opa-90))
:flex 1}) :flex 1})
(def shell-bg-container (def shell-bg-container

4
src/status_im/common/bottom_sheet/view.cljs
View File

@ -63,7 +63,7 @@
(defn view (defn view
[{:keys [hide? insets]} [{:keys [hide? insets]}
{:keys [content selected-item padding-bottom-override border-radius on-close shell? {:keys [content selected-item padding-bottom-override border-radius on-close shell?
gradient-cover? customization-color hide-handle? blur-radius blur-background] gradient-cover? customization-color hide-handle? blur-radius]
:or {border-radius 12}}] :or {border-radius 12}}]
(let [theme (quo.theme/use-theme) (let [theme (quo.theme/use-theme)
{window-height :height} (rn/get-window) {window-height :height} (rn/get-window)
@ -134,7 +134,7 @@
(when shell? (when shell?
[rn/view {:style style/shell-bg-container} [rn/view {:style style/shell-bg-container}
[quo/blur [quo/blur
{:style (style/shell-bg blur-background) {:style style/shell-bg
:blur-radius (or blur-radius 20) :blur-radius (or blur-radius 20)
:blur-amount 32 :blur-amount 32
:blur-type :transparent :blur-type :transparent

16
src/status_im/common/enter_seed_phrase/style.cljs
View File

@ -1,6 +1,5 @@
(ns status-im.common.enter-seed-phrase.style (ns status-im.common.enter-seed-phrase.style
(:require (:require [react-native.platform :as platform]))
[react-native.safe-area :as safe-area]))
(def full-layout {:flex 1}) (def full-layout {:flex 1})
@ -11,6 +10,13 @@
:left 0 :left 0
:right 0}) :right 0})
(defn recovery-phrase-container
[{:keys [banner-offset insets keyboard-shown?]}]
{:flex 1
:padding-bottom (if keyboard-shown?
(when platform/ios? banner-offset)
(:bottom insets))})
(def form-container (def form-container
{:flex 1 {:flex 1
:padding-horizontal 20 :padding-horizontal 20
@ -29,9 +35,7 @@
:margin-top 12 :margin-top 12
:margin-horizontal -20}) :margin-horizontal -20})
(defn continue-button (def continue-button
[keyboard-shown?] {:margin-top :auto})
{:margin-top :auto
:margin-bottom (when-not keyboard-shown? (safe-area/get-bottom))})
(def keyboard-container {:margin-top :auto}) (def keyboard-container {:margin-top :auto})

21
src/status_im/common/enter_seed_phrase/view.cljs
View File

@ -96,7 +96,7 @@
(take 7))) (take 7)))
(defn recovery-phrase-screen (defn recovery-phrase-screen
[{:keys [keypair title recovering-keypair? render-controls]}] [{:keys [banner-offset initial-insets keypair title recovering-keypair? render-controls]}]
(reagent/with-let [keyboard-shown? (reagent/atom false) (reagent/with-let [keyboard-shown? (reagent/atom false)
keyboard-show-listener (.addListener rn/keyboard keyboard-show-listener (.addListener rn/keyboard
"keyboardDidShow" "keyboardDidShow"
@ -114,7 +114,8 @@
seed-phrase (reagent/atom "") seed-phrase (reagent/atom "")
on-change-seed-phrase (fn [new-phrase] on-change-seed-phrase (fn [new-phrase]
(when @invalid-seed-phrase? (when @invalid-seed-phrase?
(reset! invalid-seed-phrase? false) (reset! invalid-seed-phrase? false))
(when @incorrect-seed-phrase?
(reset! incorrect-seed-phrase? false)) (reset! incorrect-seed-phrase? false))
(reset! seed-phrase new-phrase)) (reset! seed-phrase new-phrase))
on-submit (fn [] on-submit (fn []
@ -161,7 +162,10 @@
button-disabled? (or error-state? button-disabled? (or error-state?
(not (constants/seed-phrase-valid-length word-count)) (not (constants/seed-phrase-valid-length word-count))
(not all-words-valid?))] (not all-words-valid?))]
[:<> [rn/view
{:style (style/recovery-phrase-container {:insets initial-insets
:banner-offset banner-offset
:keyboard-shown? @keyboard-shown?})}
[recovery-phrase-form [recovery-phrase-form
{:title title {:title title
:keypair keypair :keypair keypair
@ -172,18 +176,18 @@
(if (fn? render-controls) (if (fn? render-controls)
(render-controls {:submit-disabled? button-disabled? (render-controls {:submit-disabled? button-disabled?
:keyboard-shown? @keyboard-shown? :keyboard-shown? @keyboard-shown?
:container-style (style/continue-button @keyboard-shown?) :container-style style/continue-button
:prepare-seed-phrase secure-clean-seed-phrase :prepare-seed-phrase secure-clean-seed-phrase
:focus-input focus-input :focus-input focus-input
:seed-phrase (security/mask-data @seed-phrase) :seed-phrase (security/mask-data @seed-phrase)
:set-incorrect-seed-phrase set-incorrect-seed-phrase}) :set-incorrect-seed-phrase set-incorrect-seed-phrase})
[quo/button [quo/button
{:container-style (style/continue-button @keyboard-shown?) {:container-style style/continue-button
:type :primary :type :primary
:disabled? button-disabled? :disabled? button-disabled?
:on-press on-submit} :on-press on-submit}
(i18n/label :t/continue)])] (i18n/label :t/continue)])]
(when @keyboard-shown? (when (or @keyboard-shown? error-state?)
[rn/view {:style style/keyboard-container} [rn/view {:style style/keyboard-container}
[quo/predictive-keyboard [quo/predictive-keyboard
{:type suggestions-state {:type suggestions-state
@ -197,7 +201,8 @@
(defn screen (defn screen
[{:keys [initial-insets title keypair navigation-icon recovering-keypair? render-controls]}] [{:keys [initial-insets title keypair navigation-icon recovering-keypair? render-controls]}]
(let [{navigation-bar-top :top} initial-insets] (let [{navigation-bar-top :top} initial-insets
banner-offset (rf/sub [:alert-banners/top-margin])]
[rn/view {:style style/full-layout} [rn/view {:style style/full-layout}
[rn/keyboard-avoiding-view {:style style/page-container} [rn/keyboard-avoiding-view {:style style/page-container}
[quo/page-nav [quo/page-nav
@ -210,6 +215,8 @@
{:title title {:title title
:keypair keypair :keypair keypair
:render-controls render-controls :render-controls render-controls
:banner-offset banner-offset
:initial-insets initial-insets
:recovering-keypair? recovering-keypair?}]]])) :recovering-keypair? recovering-keypair?}]]]))
(defn view (defn view

6
src/status_im/common/scan_qr_code/view.cljs
View File

@ -195,7 +195,7 @@
true) true)
(defn view (defn view
[{:keys [title subtitle validate-fn on-success-scan error-message share-button?]}] [{:keys [title subtitle validate-fn on-success-scan error-message share-button? import-keypair?]}]
(let [insets (safe-area/get-insets) (let [insets (safe-area/get-insets)
qr-code-succeed? (reagent/atom false) qr-code-succeed? (reagent/atom false)
qr-view-finder (reagent/atom {}) qr-view-finder (reagent/atom {})
@ -233,7 +233,9 @@
:set-qr-code-succeeded (fn [value] :set-qr-code-succeeded (fn [value]
(when on-success-scan (when on-success-scan
(on-success-scan value)) (on-success-scan value))
(rf/dispatch [:navigate-back])) (if import-keypair?
(set-rescan-timeout)
(rf/dispatch [:navigate-back])))
:set-rescan-timeout set-rescan-timeout}]) :set-rescan-timeout set-rescan-timeout}])
[rn/view {:style (style/root-container (:top insets))} [rn/view {:style (style/root-container (:top insets))}
[header [header

1
src/status_im/common/standard_authentication/enter_password/view.cljs
View File

@ -33,6 +33,7 @@
:size 24}]] :size 24}]]
[password-input/view [password-input/view
{:on-press-biometrics on-press-biometrics {:on-press-biometrics on-press-biometrics
:blur? true
:processing processing :processing processing
:error error :error error
:default-password password :default-password password

7
src/status_im/common/standard_authentication/events.cljs
View File

@ -20,7 +20,7 @@
(rf/reg-event-fx :standard-auth/authorize authorize) (rf/reg-event-fx :standard-auth/authorize authorize)
(defn authorize-with-biometric (defn authorize-with-biometric
[_ [{:keys [on-auth-success on-auth-fail] :as args}]] [_ [{:keys [on-auth-success on-auth-fail on-close] :as args}]]
(let [args-with-biometric-btn (let [args-with-biometric-btn
(assoc args (assoc args
:on-press-biometric :on-press-biometric
@ -31,7 +31,10 @@
{:prompt-message (i18n/label :t/biometric-auth-confirm-message) {:prompt-message (i18n/label :t/biometric-auth-confirm-message)
:on-cancel #(rf/dispatch [:standard-auth/authorize-with-password :on-cancel #(rf/dispatch [:standard-auth/authorize-with-password
args-with-biometric-btn]) args-with-biometric-btn])
:on-success #(rf/dispatch [:standard-auth/on-biometric-success on-auth-success]) :on-success (fn []
(when (fn? on-close)
(on-close))
(rf/dispatch [:standard-auth/on-biometric-success on-auth-success]))
:on-fail (fn [err] :on-fail (fn [err]
(rf/dispatch [:standard-auth/authorize-with-password (rf/dispatch [:standard-auth/authorize-with-password
args-with-biometric-btn]) args-with-biometric-btn])

8
src/status_im/common/validation/keypair.cljs
View File

@ -14,10 +14,12 @@
(> (-> s str string/trim count) constants/key-pair-name-max-length)) (> (-> s str string/trim count) constants/key-pair-name-max-length))
(defn validation-keypair-name (defn validation-keypair-name
[s] [s existing-keypair-names]
(cond (cond
(string/blank? s) nil (string/blank? s) nil
(validators/has-emojis? s) (i18n/label :t/key-name-error-emoji) (validators/has-emojis? s) (i18n/label :t/key-name-error-emoji)
(validators/has-special-characters? s) (i18n/label :t/key-name-error-special-char) (validators/has-special-characters? s) (i18n/label :t/key-name-error-special-char)
(keypair-too-short? s) (i18n/label :t/your-key-pair-name-is-too-short) (keypair-too-short? s) (i18n/label :t/key-name-error-too-short
(keypair-too-long? s) (i18n/label :t/your-key-pair-name-is-too-long))) {:count constants/key-pair-name-min-length})
(keypair-too-long? s) (i18n/label :t/your-key-pair-name-is-too-long)
(contains? existing-keypair-names s) (i18n/label :t/key-name-error-taken)))

7
src/status_im/common/validation/keypair_test.cljs
View File

@ -2,6 +2,7 @@
(:require (:require
[cljs.test :refer-macros [deftest are]] [cljs.test :refer-macros [deftest are]]
[status-im.common.validation.keypair :as keypair-validator] [status-im.common.validation.keypair :as keypair-validator]
[status-im.constants :as constants]
[utils.i18n :as i18n])) [utils.i18n :as i18n]))
(deftest keypair-name-too-short-test (deftest keypair-name-too-short-test
@ -18,10 +19,12 @@
(deftest validation-keypair-name-test (deftest validation-keypair-name-test
(are [arg expected] (are [arg expected]
(= (keypair-validator/validation-keypair-name arg) expected) (= (keypair-validator/validation-keypair-name arg #{"Collection"}) expected)
nil nil nil nil
"" nil "" nil
"name !" (i18n/label :t/key-name-error-special-char) "name !" (i18n/label :t/key-name-error-special-char)
"Hello 😊" (i18n/label :t/key-name-error-emoji) "Hello 😊" (i18n/label :t/key-name-error-emoji)
"abc" (i18n/label :t/your-key-pair-name-is-too-short) "abc" (i18n/label :t/key-name-error-too-short
{:count constants/key-pair-name-min-length})
"Collection" (i18n/label :t/key-name-error-taken)
(apply str (repeat 25 "a")) (i18n/label :t/your-key-pair-name-is-too-long))) (apply str (repeat 25 "a")) (i18n/label :t/your-key-pair-name-is-too-long)))

20
src/status_im/contexts/settings/wallet/events.cljs
View File

@ -92,23 +92,28 @@
(rf/reg-event-fx :wallet/make-keypairs-accounts-fully-operable make-keypairs-accounts-fully-operable) (rf/reg-event-fx :wallet/make-keypairs-accounts-fully-operable make-keypairs-accounts-fully-operable)
(defn connection-string-for-import-keypair (defn connection-string-for-import-keypair
[{:keys [db]} [{:keys [sha3-pwd keypairs-key-uids connection-string]}]] [{:keys [db]} [{:keys [sha3-pwd keypairs-key-uids connection-string on-success]}]]
(let [key-uid (get-in db [:profile/profile :key-uid])] (let [key-uid (get-in db [:profile/profile :key-uid])]
{:fx [[:effects.syncing/import-keypairs-keystores {:fx [[:effects.syncing/import-keypairs-keystores
{:key-uid key-uid {:key-uid key-uid
:sha3-pwd sha3-pwd :sha3-pwd sha3-pwd
:keypairs-key-uids keypairs-key-uids :keypairs-key-uids keypairs-key-uids
:connection-string connection-string :connection-string connection-string
:on-success #(rf/dispatch [:wallet/make-keypairs-accounts-fully-operable %]) :on-success (fn [key-uids]
:on-fail #(rf/dispatch [:toasts/upsert (rf/call-continuation on-success)
(rf/dispatch [:wallet/make-keypairs-accounts-fully-operable key-uids]))
:on-fail (fn [error]
(log/error "failed to import missing key pairs with connection string"
{:error error})
(rf/dispatch [:toasts/upsert
{:type :negative {:type :negative
:theme :dark :theme :dark
:text %}])}]]})) :text (i18n/label :t/incorrect-qr-code)}]))}]]}))
(rf/reg-event-fx :wallet/connection-string-for-import-keypair connection-string-for-import-keypair) (rf/reg-event-fx :wallet/connection-string-for-import-keypair connection-string-for-import-keypair)
(defn success-keypair-qr-scan (defn success-keypair-qr-scan
[_ [connection-string keypairs-key-uids]] [_ [connection-string keypairs-key-uids on-import-success]]
{:fx [[:dispatch {:fx [[:dispatch
[:standard-auth/authorize-with-password [:standard-auth/authorize-with-password
{:blur? true {:blur? true
@ -120,7 +125,8 @@
[:wallet/connection-string-for-import-keypair [:wallet/connection-string-for-import-keypair
{:connection-string connection-string {:connection-string connection-string
:keypairs-key-uids keypairs-key-uids :keypairs-key-uids keypairs-key-uids
:sha3-pwd password}]))}]]]}) :sha3-pwd password
:on-success on-import-success}]))}]]]})
(rf/reg-event-fx :wallet/success-keypair-qr-scan success-keypair-qr-scan) (rf/reg-event-fx :wallet/success-keypair-qr-scan success-keypair-qr-scan)
@ -146,7 +152,7 @@
{:fx [[:json-rpc/call {:fx [[:json-rpc/call
[{:method "accounts_makePrivateKeyKeypairFullyOperable" [{:method "accounts_makePrivateKeyKeypairFullyOperable"
:params [(security/safe-unmask-data private-key) :params [(security/safe-unmask-data private-key)
(-> password security/safe-unmask-data native-module/sha3)] (security/safe-unmask-data password)]
:on-success on-success :on-success on-success
:on-error on-error}]]]}) :on-error on-error}]]]})

18
src/status_im/contexts/settings/wallet/keypairs_and_accounts/actions/view.cljs
View File

@ -41,31 +41,37 @@
[quo/action-drawer [quo/action-drawer
[(when has-paired-device [(when has-paired-device
(if-not missing-keypair? (if-not missing-keypair?
[{:icon :i/qr-code [{:blur? true
:icon :i/qr-code
:accessibility-label :show-key-pr-qr :accessibility-label :show-key-pr-qr
:label (i18n/label :t/show-encrypted-qr-of-key-pairs) :label (i18n/label :t/show-encrypted-qr-of-key-pairs)
:on-press on-show-qr}] :on-press on-show-qr}]
[{:icon :i/scan [{:blur? true
:icon :i/scan
:accessibility-label :import-by-scan-qr :accessibility-label :import-by-scan-qr
:label (i18n/label :t/import-by-scanning-encrypted-qr) :label (i18n/label :t/import-by-scanning-encrypted-qr)
:on-press on-scan-qr}])) :on-press on-scan-qr}]))
(when (= (:type drawer-props) :keypair) (when (= (:type drawer-props) :keypair)
[(when missing-keypair? [(when missing-keypair?
(case (:type keypair) (case (:type keypair)
:seed {:icon :i/seed :seed {:blur? true
:icon :i/seed
:accessibility-label :import-seed-phrase :accessibility-label :import-seed-phrase
:label (i18n/label :t/import-by-entering-recovery-phrase) :label (i18n/label :t/import-by-entering-recovery-phrase)
:on-press on-import-seed-phrase} :on-press on-import-seed-phrase}
:key {:icon :i/key :key {:blur? true
:icon :i/key
:accessibility-label :import-private-key :accessibility-label :import-private-key
:label (i18n/label :t/import-by-entering-private-key) :label (i18n/label :t/import-by-entering-private-key)
:on-press on-import-private-key} :on-press on-import-private-key}
nil)) nil))
{:icon :i/edit {:blur? true
:icon :i/edit
:accessibility-label :rename-key-pair :accessibility-label :rename-key-pair
:label (i18n/label :t/rename-key-pair) :label (i18n/label :t/rename-key-pair)
:on-press on-rename-keypair} :on-press on-rename-keypair}
{:icon :i/delete {:blur? true
:icon :i/delete
:accessibility-label :remove-key-pair :accessibility-label :remove-key-pair
:add-divider? true :add-divider? true
:danger? true :danger? true

9
src/status_im/contexts/settings/wallet/keypairs_and_accounts/missing_keypairs/scan_qr/view.cljs
View File

@ -11,13 +11,16 @@
[] []
(let [keypairs-key-uids (rf/sub [:get-screen-params]) (let [keypairs-key-uids (rf/sub [:get-screen-params])
on-success-scan (rn/use-callback (fn [scanned-text] on-success-scan (rn/use-callback (fn [scanned-text]
(rf/dispatch [:wallet/success-keypair-qr-scan scanned-text (rf/dispatch [:wallet/success-keypair-qr-scan
keypairs-key-uids]) scanned-text
keypairs-key-uids
[:navigate-back]])
[keypairs-key-uids]))] [keypairs-key-uids]))]
[scan-qr-code/view [scan-qr-code/view
{:title (i18n/label :t/scan-key-pairs-qr-code) {:title (i18n/label :t/scan-key-pairs-qr-code)
:subtitle (i18n/label :t/find-it-in-setting) :subtitle (i18n/label :t/find-it-in-setting)
:share-button? false :share-button? false
:import-keypair? true
:validate-fn sync-utils/valid-connection-string? :validate-fn sync-utils/valid-connection-string?
:error-message (i18n/label :t/invalid-qr) :error-message (i18n/label :t/invalid-key-pair-qr)
:on-success-scan on-success-scan}])) :on-success-scan on-success-scan}]))

3
src/status_im/contexts/settings/wallet/keypairs_and_accounts/rename/style.cljs
View File

@ -4,7 +4,8 @@
{:margin-bottom 8}) {:margin-bottom 8})
(def bottom-action (def bottom-action
{:margin-horizontal -20}) {:margin-horizontal -20
:margin-vertical -12})
(def error-container (def error-container
{:margin-left 20 {:margin-left 20

46
src/status_im/contexts/settings/wallet/keypairs_and_accounts/rename/view.cljs
View File

@ -2,6 +2,7 @@
(:require [clojure.string :as string] (:require [clojure.string :as string]
[quo.core :as quo] [quo.core :as quo]
[react-native.core :as rn] [react-native.core :as rn]
[react-native.safe-area :as safe-area]
[status-im.common.floating-button-page.view :as floating-button-page] [status-im.common.floating-button-page.view :as floating-button-page]
[status-im.common.validation.keypair :as keypair-validator] [status-im.common.validation.keypair :as keypair-validator]
[status-im.constants :as constants] [status-im.constants :as constants]
@ -14,19 +15,24 @@
(defn view (defn view
[] []
(let [{:keys [name key-uid]} (rf/sub [:get-screen-params]) (let [insets (safe-area/get-insets)
{existing-keypair-name :name
:keys [key-uid]} (rf/sub [:get-screen-params])
existing-keypair-names (rf/sub [:wallet/keypair-names])
customization-color (rf/sub [:profile/customization-color]) customization-color (rf/sub [:profile/customization-color])
[unsaved-keypair-name set-unsaved-keypair-name] (rn/use-state name) [unsaved-keypair-name set-unsaved-keypair-name] (rn/use-state existing-keypair-name)
[error-msg set-error-msg] (rn/use-state nil) [error-msg set-error-msg] (rn/use-state nil)
[typing? set-typing?] (rn/use-state false) [typing? set-typing?] (rn/use-state false)
validate-keypair-name (rn/use-callback validate-keypair-name (rn/use-callback
(debounce/debounce (debounce/debounce
(fn [name] (fn [input-name]
(set-error-msg (set-error-msg
(keypair-validator/validation-keypair-name (keypair-validator/validation-keypair-name
name)) input-name
existing-keypair-names))
(set-typing? false)) (set-typing? false))
300)) 300)
[existing-keypair-names])
on-change-text (rn/use-callback (fn [text] on-change-text (rn/use-callback (fn [text]
(set-typing? true) (set-typing? true)
(set-unsaved-keypair-name (set-unsaved-keypair-name
@ -40,17 +46,28 @@
:keypair-name :keypair-name
unsaved-keypair-name}]) unsaved-keypair-name}])
[unsaved-keypair-name key-uid])] [unsaved-keypair-name key-uid])]
[quo/overlay {:type :shell}
[floating-button-page/view [floating-button-page/view
{:header [quo/page-nav {:footer-container-padding 0
{:icon-name :i/close :blur? true
:header [quo/page-nav
{:margin-top (:top insets)
:icon-name :i/close
:background :blur
:on-press navigate-back :on-press navigate-back
:accessibility-label :top-bar}] :accessibility-label :top-bar}]
:footer [quo/bottom-actions :footer [quo/bottom-actions
{:actions :one-action {:actions :one-action
:blur? true
:button-one-label (i18n/label :t/save) :button-one-label (i18n/label :t/save)
:button-one-props {:disabled? (or typing? :button-one-props {:blur? true
(string/blank? unsaved-keypair-name) :disabled? (or typing?
(not (string/blank? error-msg))) (= existing-keypair-name
unsaved-keypair-name)
(string/blank?
unsaved-keypair-name)
(not (string/blank?
error-msg)))
:customization-color customization-color :customization-color customization-color
:on-press on-save} :on-press on-save}
:container-style style/bottom-action}]} :container-style style/bottom-action}]}
@ -58,17 +75,18 @@
{:container-style style/header-container {:container-style style/header-container
:title (i18n/label :t/rename-key-pair) :title (i18n/label :t/rename-key-pair)
:description :context-tag :description :context-tag
:blur? true
:context-tag {:type :icon :context-tag {:type :icon
:size 24 :size 24
:context name :context existing-keypair-name
:icon :i/seed-phrase}}] :icon :i/seed-phrase}}]
[quo/input [quo/input
{:container-style {:margin-horizontal 20} {:blur? true
:container-style {:margin-horizontal 20}
:placeholder (i18n/label :t/keypair-name-input-placeholder) :placeholder (i18n/label :t/keypair-name-input-placeholder)
:label (i18n/label :t/keypair-name) :label (i18n/label :t/keypair-name)
:default-value unsaved-keypair-name :default-value unsaved-keypair-name
:char-limit constants/key-pair-name-max-length :char-limit constants/key-pair-name-max-length
:max-length constants/key-pair-name-max-length
:auto-focus true :auto-focus true
:clearable? (not (string/blank? unsaved-keypair-name)) :clearable? (not (string/blank? unsaved-keypair-name))
:on-clear on-clear :on-clear on-clear
@ -80,5 +98,5 @@
:size :default :size :default
:icon :i/info :icon :i/info
:container-style style/error-container} :container-style style/error-container}
error-msg])])) error-msg])]]))

7
src/status_im/contexts/settings/wallet/keypairs_and_accounts/view.cljs
View File

@ -1,11 +1,11 @@
(ns status-im.contexts.settings.wallet.keypairs-and-accounts.view (ns status-im.contexts.settings.wallet.keypairs-and-accounts.view
(:require [quo.core :as quo] (:require [quo.core :as quo]
[quo.foundations.colors :as colors]
[quo.theme] [quo.theme]
[react-native.core :as rn] [react-native.core :as rn]
[react-native.safe-area :as safe-area] [react-native.safe-area :as safe-area]
[status-im.contexts.settings.wallet.keypairs-and-accounts.actions.view :as actions] [status-im.contexts.settings.wallet.keypairs-and-accounts.actions.view :as actions]
[status-im.contexts.settings.wallet.keypairs-and-accounts.style :as style] [status-im.contexts.settings.wallet.keypairs-and-accounts.style :as style]
[status-im.feature-flags :as ff]
[utils.address :as utils] [utils.address :as utils]
[utils.i18n :as i18n] [utils.i18n :as i18n]
[utils.re-frame :as rf])) [utils.re-frame :as rf]))
@ -21,7 +21,6 @@
{:drawer-props drawer-props {:drawer-props drawer-props
:keypair keypair}]) :keypair keypair}])
:blur-background colors/bottom-sheet-background-blur
:theme (:theme drawer-props) :theme (:theme drawer-props)
:shell? true}])) :shell? true}]))
@ -70,7 +69,7 @@
{:blur? true {:blur? true
:status-indicator false :status-indicator false
:stored :on-device :stored :on-device
:action :options :action (if default-keypair? :none :options)
:accounts accounts :accounts accounts
:customization-color customization-color :customization-color customization-color
:container-style style/keypair-container-style :container-style style/keypair-container-style
@ -85,7 +84,6 @@
(rf/dispatch [:show-bottom-sheet (rf/dispatch [:show-bottom-sheet
{:theme :dark {:theme :dark
:shell? true :shell? true
:blur-background colors/bottom-sheet-background-blur
:content (fn [] [actions/view :content (fn [] [actions/view
{:keypair keypair-data {:keypair keypair-data
:drawer-props (options-drawer-props :drawer-props (options-drawer-props
@ -127,6 +125,7 @@
:header (when (seq missing-keypairs) :header (when (seq missing-keypairs)
[quo/missing-keypairs [quo/missing-keypairs
{:blur? true {:blur? true
:show-import-all? (ff/enabled? ::ff/settings.import-all-keypairs)
:keypairs missing-keypairs :keypairs missing-keypairs
:on-import-press on-import-press :on-import-press on-import-press
:container-style style/missing-keypairs-container-style :container-style style/missing-keypairs-container-style

4
src/status_im/contexts/settings/wallet/network_settings/view.cljs
View File

@ -1,6 +1,5 @@
(ns status-im.contexts.settings.wallet.network-settings.view (ns status-im.contexts.settings.wallet.network-settings.view
(:require [quo.core :as quo] (:require [quo.core :as quo]
[quo.foundations.colors :as colors]
[quo.foundations.resources :as resources] [quo.foundations.resources :as resources]
[quo.theme] [quo.theme]
[react-native.core :as rn] [react-native.core :as rn]
@ -84,8 +83,7 @@
{:enable? enable? {:enable? enable?
:blur? blur?}]) :blur? blur?}])
:theme theme :theme theme
:shell? blur? :shell? blur?}]))
:blur-background colors/bottom-sheet-background-blur}]))
(defn view (defn view
[] []

2
src/status_im/contexts/settings/wallet/saved_addresses/sheets/address_options/view.cljs
View File

@ -2,7 +2,6 @@
(:require (:require
[clojure.string :as string] [clojure.string :as string]
[quo.core :as quo] [quo.core :as quo]
[quo.foundations.colors :as colors]
[react-native.core :as rn] [react-native.core :as rn]
[react-native.platform :as platform] [react-native.platform :as platform]
[status-im.constants :as constants] [status-im.constants :as constants]
@ -54,7 +53,6 @@
[:show-bottom-sheet [:show-bottom-sheet
{:theme :dark {:theme :dark
:shell? true :shell? true
:blur-background colors/bottom-sheet-background-blur
:content (fn [] :content (fn []
[remove-address/view opts])}]) [remove-address/view opts])}])
[opts]) [opts])

9
src/status_im/contexts/settings/wallet/wallet_options/view.cljs
View File

@ -16,11 +16,10 @@
(defn basic-settings-options (defn basic-settings-options
[] []
[(when (ff/enabled? ::ff/settings.keypairs-and-accounts) [{:title (i18n/label :t/keypairs-and-accounts)
{:title (i18n/label :t/keypairs-and-accounts)
:blur? true :blur? true
:on-press open-keypairs-and-accounts-settings-modal :on-press open-keypairs-and-accounts-settings-modal
:action :arrow}) :action :arrow}
(when (ff/enabled? ::ff/settings.saved-addresses) (when (ff/enabled? ::ff/settings.saved-addresses)
{:title (i18n/label :t/saved-addresses) {:title (i18n/label :t/saved-addresses)
:blur? true :blur? true
@ -74,7 +73,5 @@
[quo/page-top [quo/page-top
{:title (i18n/label :t/wallet) {:title (i18n/label :t/wallet)
:title-accessibility-label :wallet-settings-header}] :title-accessibility-label :wallet-settings-header}]
(when (or (ff/enabled? ::ff/settings.keypairs-and-accounts) [basic-settings]
(ff/enabled? ::ff/settings.saved-addresses))
[basic-settings])
[advanced-settings]])) [advanced-settings]]))

2
src/status_im/contexts/wallet/effects.cljs
View File

@ -73,7 +73,7 @@
(fn [resolver rejecter] (fn [resolver rejecter]
(json-rpc/call {:method "accounts_makeSeedPhraseKeypairFullyOperable" (json-rpc/call {:method "accounts_makeSeedPhraseKeypairFullyOperable"
:params [(security/safe-unmask-data mnemonic) :params [(security/safe-unmask-data mnemonic)
(-> password security/safe-unmask-data native-module/sha3)] (security/safe-unmask-data password)]
:on-error (fn [error] :on-error (fn [error]
(rejecter (ex-info (str error) {:error error}))) (rejecter (ex-info (str error) {:error error})))
:on-success (fn [value] :on-success (fn [value]

13
src/status_im/contexts/wallet/events.cljs
View File

@ -529,18 +529,17 @@
:wallet/process-keypair-from-backup :wallet/process-keypair-from-backup
(fn [{:keys [db]} [{:keys [backedUpKeypair]}]] (fn [{:keys [db]} [{:keys [backedUpKeypair]}]]
(let [{:keys [key-uid accounts]} backedUpKeypair (let [{:keys [key-uid accounts]} backedUpKeypair
updated-keypairs (assoc-in db accounts-fx
[:wallet :keypairs key-uid] (mapv (fn [{:keys [chat] :as account}]
(data-store/rpc->keypair backedUpKeypair)) ;; We exclude the chat account from the profile keypair for fetching the assets
accounts-fx (mapv (fn [{:keys [chat] :as account}]
;; We exclude the chat account from the profile keypair
;; for fetching the assets
(when-not chat (when-not chat
[:dispatch [:dispatch
[:wallet/process-account-from-signal [:wallet/process-account-from-signal
account]])) account]]))
accounts)] accounts)]
{:db (assoc-in db [:wallet :keypairs] updated-keypairs) {:db (assoc-in db
[:wallet :keypairs key-uid]
(data-store/rpc->keypair backedUpKeypair))
:fx accounts-fx}))) :fx accounts-fx})))
(rf/reg-event-fx (rf/reg-event-fx

4
src/status_im/feature_flags.cljs
View File

@ -11,10 +11,10 @@
(def ^:private initial-flags (def ^:private initial-flags
{::community.edit-account-selection (enabled-in-env? :FLAG_EDIT_ACCOUNT_SELECTION_ENABLED) {::community.edit-account-selection (enabled-in-env? :FLAG_EDIT_ACCOUNT_SELECTION_ENABLED)
::settings.keypairs-and-accounts (enabled-in-env?
:FLAG_WALLET_SETTINGS_KEYPAIRS_AND_ACCOUNTS_ENABLED)
::settings.saved-addresses (enabled-in-env? ::settings.saved-addresses (enabled-in-env?
:FLAG_WALLET_SETTINGS_SAVED_ADDRESSES_ENABLED) :FLAG_WALLET_SETTINGS_SAVED_ADDRESSES_ENABLED)
::settings.import-all-keypairs (enabled-in-env?
:FLAG_WALLET_SETTINGS_IMPORT_ALL_KEYPAIRS)
::shell.jump-to (enabled-in-env? :ENABLE_JUMP_TO) ::shell.jump-to (enabled-in-env? :ENABLE_JUMP_TO)
::wallet.advanced-sending (enabled-in-env? :FLAG_ADVANCED_SENDING) ::wallet.advanced-sending (enabled-in-env? :FLAG_ADVANCED_SENDING)
::wallet.assets-modal-hide (enabled-in-env? :FLAG_ASSETS_MODAL_HIDE) ::wallet.assets-modal-hide (enabled-in-env? :FLAG_ASSETS_MODAL_HIDE)

4
src/status_im/navigation/screens.cljs
View File

@ -542,7 +542,7 @@
:component wallet-options/view} :component wallet-options/view}
{:name :screen/settings.rename-keypair {:name :screen/settings.rename-keypair
:options (assoc options/dark-screen :sheet? true) :options options/transparent-screen-options
:component keypair-rename/view} :component keypair-rename/view}
{:name :screen/settings.encrypted-keypair-qr {:name :screen/settings.encrypted-keypair-qr
@ -558,7 +558,7 @@
:component keypairs-and-accounts/view} :component keypairs-and-accounts/view}
{:name :screen/settings.scan-keypair-qr {:name :screen/settings.scan-keypair-qr
:options options/transparent-modal-screen-options :options options/transparent-screen-options
:component scan-keypair-qr/view} :component scan-keypair-qr/view}
{:name :screen/settings.missing-keypair.import-seed-phrase {:name :screen/settings.missing-keypair.import-seed-phrase

8
src/status_im/subs/wallet/wallet.cljs
View File

@ -205,6 +205,12 @@
(fn [{:keys [keypairs]}] (fn [{:keys [keypairs]}]
(vals keypairs))) (vals keypairs)))
(rf/reg-sub
:wallet/keypair-names
:<- [:wallet/keypairs-list]
(fn [keypairs]
(set (map :name keypairs))))
(rf/reg-sub (rf/reg-sub
:wallet/selected-keypair-uid :wallet/selected-keypair-uid
:<- [:wallet/create-account] :<- [:wallet/create-account]
@ -244,7 +250,7 @@
size 32}}] size 32}}]
(->> accounts (->> accounts
(keep (fn [{:keys [path color emoji name address]}] (keep (fn [{:keys [path color emoji name address]}]
(when-not (string/starts-with? path constants/path-eip1581) (when-not (string/starts-with? (str path) constants/path-eip1581)
{:account-props {:customization-color color {:account-props {:customization-color color
:size size :size size
:emoji emoji :emoji emoji

14
src/status_im/subs/wallet/wallet_test.cljs
View File

@ -643,18 +643,20 @@
:removed false}) :removed false})
(def profile-key-pair-key-uid "abc") (def profile-key-pair-key-uid "abc")
(def profile-key-pair-name "My Profile")
(def seed-phrase-key-pair-key-uid "def") (def seed-phrase-key-pair-key-uid "def")
(def seed-phrase-key-pair-name "My Key Pair")
(def profile-keypair (def profile-keypair
{:key-uid profile-key-pair-key-uid {:key-uid profile-key-pair-key-uid
:name "My Profile" :name profile-key-pair-name
:type :profile :type :profile
:lowest-operability :fully :lowest-operability :fully
:accounts []}) :accounts []})
(def seed-phrase-keypair (def seed-phrase-keypair
{:key-uid seed-phrase-key-pair-key-uid {:key-uid seed-phrase-key-pair-key-uid
:name "My Key Pair" :name seed-phrase-key-pair-name
:type :seed :type :seed
:lowest-operability :no :lowest-operability :no
:accounts []}) :accounts []})
@ -676,6 +678,14 @@
(is (= 2 (count result))) (is (= 2 (count result)))
(is (match? expected result)))) (is (match? expected result))))
(h/deftest-sub :wallet/keypair-names
[sub-name]
(swap! rf-db/app-db assoc-in
[:wallet :keypairs]
{profile-key-pair-key-uid profile-keypair
seed-phrase-key-pair-key-uid seed-phrase-keypair})
(is (match? #{seed-phrase-key-pair-name profile-key-pair-name} (rf/sub [sub-name]))))
(h/deftest-sub :wallet/settings-keypairs-accounts (h/deftest-sub :wallet/settings-keypairs-accounts
[sub-name] [sub-name]
(testing "returns formatted key-pairs and accounts" (testing "returns formatted key-pairs and accounts"

3
translations/en.json
View File

@ -1302,6 +1302,8 @@
"scan-with-status-app": "Scan with the Status app on another device", "scan-with-status-app": "Scan with the Status app on another device",
"scan-key-pairs-qr-code": "Scan key pairs QR code", "scan-key-pairs-qr-code": "Scan key pairs QR code",
"invalid-qr": "Oops! This QR doesn’t work with Status", "invalid-qr": "Oops! This QR doesn’t work with Status",
"invalid-key-pair-qr": "This does not look like a key pair QR code",
"incorrect-qr-code": "This is not the QR code you are looking for",
"search": "Search", "search": "Search",
"search-discover-communities": "Search communities or categories", "search-discover-communities": "Search communities or categories",
"secret-keys-confirmation-text": "You will need them to continue to use your Keycard in case you ever lose your phone.", "secret-keys-confirmation-text": "You will need them to continue to use your Keycard in case you ever lose your phone.",
@ -2639,6 +2641,7 @@
"max": "Max: {{number}}", "max": "Max: {{number}}",
"your-key-pair-name-is-too-long": "Your key pair name is too long", "your-key-pair-name-is-too-long": "Your key pair name is too long",
"your-key-pair-name-is-too-short": "Your key pair name is too short", "your-key-pair-name-is-too-short": "Your key pair name is too short",
"key-name-error-taken": "Key pair name already in use",
"key-name-error-length": "Key name too long", "key-name-error-length": "Key name too long",
"key-name-error-emoji": "Emojis are not allowed", "key-name-error-emoji": "Emojis are not allowed",
"key-name-error-special-char": "Special characters are not allowed", "key-name-error-special-char": "Special characters are not allowed",