Skip to content

Commit b177f55

Browse files
committed
events added
1 parent 6d14dc6 commit b177f55

File tree

6 files changed

+40
-41
lines changed

6 files changed

+40
-41
lines changed

src/status_im/contexts/centralized_metrics/events_test.cljs

+1-1
Original file line numberDiff line numberDiff line change
@@ -15,7 +15,7 @@
1515

1616
(deftest centralized-metrics-interceptor-test
1717
(testing "processes context correctly"
18-
(with-redefs [tracking/tracked-event (fn [_] {:metric "mocked-event"})
18+
(with-redefs [tracking/tracked-event (fn [_ _] {:metric "mocked-event"})
1919
events/push-event? (fn [_] true)]
2020
(let [context {:coeffects {:event [:some-event]
2121
:db {:centralized-metrics/enabled? true}}}]

src/status_im/contexts/centralized_metrics/tracking.cljs

+19-34
Original file line numberDiff line numberDiff line change
@@ -53,9 +53,21 @@
5353
(when (contains? view-ids-to-track view-id)
5454
(navigation-event (name view-id))))
5555

56+
(defn collectilbes-fetched-event
57+
[db]
58+
(let [accounts (get-in db [:wallet :accounts])
59+
amount-on-all-accounts (reduce (fn [collectibles-amount account]
60+
(+ collectibles-amount (:current-collectible-idx account)))
61+
0
62+
(vals accounts))]
63+
(key-value-event "collectibles-fetched" :total-amount amount-on-all-accounts)))
64+
65+
(defn navigated-to-collectibles-tab-event
66+
[location]
67+
(key-value-event "navigated-to-collectibles-tab" :location location))
68+
5669
(defn tracked-event
5770
[[event-name second-parameter] db]
58-
(def dbg-db db)
5971
(case event-name
6072
:profile/get-profiles-overview-success
6173
(user-journey-event app-started-event)
@@ -68,40 +80,13 @@
6880

6981
:wallet/select-account-tab
7082
(when (= second-parameter :collectibles)
71-
(key-value-event "navigated-to-collectibles-tab" :location :account))
83+
(navigated-to-collectibles-tab-event :account))
84+
85+
:wallet/select-home-tab
86+
(when (= second-parameter :collectibles)
87+
(navigated-to-collectibles-tab-event :home))
7288

7389
:wallet/flush-collectibles-fetched
74-
(let [accounts (get-in db [:wallet :accounts])
75-
total-amount (reduce (fn [collectibles-amount account]
76-
(+ collectibles-amount (:current-collectible-idx account)))
77-
0
78-
(vals accounts))]
79-
(key-value-event "collectibles-fetched" :total-amount total-amount))
90+
(collectilbes-fetched-event db)
8091

81-
82-
;; :centralized-metrics/navigated-to-collectibles-tab
83-
;; (key-value-event "navigated-to-collectibles-tab" :location second-parameter)
84-
85-
;; :centralized-metrics/collectibles-fetched
86-
;; (key-value-event "collectibles-fetched" :amount second-parameter)
87-
8892
nil))
89-
90-
91-
(comment
92-
dbg-db
93-
(def accounts (get-in dbg-db [:wallet :accounts]))
94-
(:current-collectible-idx (second (vals accounts)))
95-
(reduce (fn [collectibles-amount account]
96-
(+ collectibles-amount (:current-collectible-idx account)))
97-
(vals accounts))
98-
99-
100-
101-
(reduce (fn [collectibles-amount account]
102-
(+ collectibles-amount (:current-collectible-idx account)))
103-
0
104-
(vals {:a {:current-collectible-idx 1}
105-
:b {:current-collectible-idx 1}
106-
})))
107-

src/status_im/contexts/centralized_metrics/tracking_test.cljs

+4-4
Original file line numberDiff line numberDiff line change
@@ -63,17 +63,17 @@
6363
:platform platform-os
6464
:appVersion app-version
6565
:eventValue {:action tracking/app-started-event}}}
66-
(tracking/tracked-event [:profile/get-profiles-overview-success])))
66+
(tracking/tracked-event [:profile/get-profiles-overview-success] {})))
6767
(is (= {:metric
6868
{:eventName "events.metrics-enabled"
6969
:platform platform-os
7070
:appVersion app-version
7171
:eventValue {:enabled true}}}
72-
(tracking/tracked-event [:centralized-metrics/toggle-centralized-metrics true])))
72+
(tracking/tracked-event [:centralized-metrics/toggle-centralized-metrics true ] {})))
7373
(is (= {:metric
7474
{:eventName "navigation"
7575
:platform platform-os
7676
:appVersion app-version
7777
:eventValue {:viewId "wallet-stack"}}}
78-
(tracking/tracked-event [:set-view-id :wallet-stack])))
79-
(is (nil? (tracking/tracked-event [:unknown-event])))))
78+
(tracking/tracked-event [:set-view-id :wallet-stack] {})))
79+
(is (nil? (tracking/tracked-event [:unknown-event] {})))))

src/status_im/contexts/wallet/events.cljs

+4
Original file line numberDiff line numberDiff line change
@@ -58,6 +58,10 @@
5858
(fn [{:keys [db]} [tab]]
5959
{:db (assoc-in db [:wallet :ui :account-page :active-tab] tab)}))
6060

61+
(rf/reg-event-fx :wallet/select-home-tab
62+
(fn [{:keys [db]} [tab]]
63+
{:db (assoc-in db [:wallet :ui :active-tab] tab)}))
64+
6165
(rf/reg-event-fx :wallet/clear-account-tab
6266
(fn [{:keys [db]}]
6367
{:db (assoc-in db [:wallet :ui :account-page :active-tab] nil)}))

src/status_im/contexts/wallet/home/view.cljs

+6-2
Original file line numberDiff line numberDiff line change
@@ -35,12 +35,16 @@
3535
:on-press #(rf/dispatch [:show-bottom-sheet {:content new-account}])
3636
:type :add-account})
3737

38+
(def first-tab-id :assets)
39+
3840
(def tabs-data
3941
[{:id :assets :label (i18n/label :t/assets) :accessibility-label :assets-tab}
4042
{:id :collectibles :label (i18n/label :t/collectibles) :accessibility-label :collectibles-tab}
4143
(when (ff/enabled? ::ff/wallet.home-activity)
4244
{:id :activity :label (i18n/label :t/activity) :accessibility-label :activity-tab})])
4345

46+
(defn- change-tab [id] (rf/dispatch [:wallet/select-home-tab id]))
47+
4448
(defn- render-cards
4549
[cards ref]
4650
[rn/flat-list
@@ -64,7 +68,7 @@
6468

6569
(defn view
6670
[]
67-
(let [[selected-tab set-selected-tab] (rn/use-state (:id (first tabs-data)))
71+
(let [ selected-tab (or (rf/sub [:wallet/home-tab]) first-tab-id)
6872
account-list-ref (rn/use-ref-atom nil)
6973
tokens-loading? (rf/sub [:wallet/home-tokens-loading?])
7074
networks (rf/sub [:wallet/selected-network-details])
@@ -104,7 +108,7 @@
104108
(when (ff/enabled? ::ff/wallet.graph)
105109
[quo/wallet-graph {:time-frame :empty}])
106110
[render-cards cards account-list-ref]
107-
[render-tabs tabs-data set-selected-tab selected-tab]]
111+
[render-tabs tabs-data change-tab selected-tab]]
108112
:content-container-style style/list-container
109113
:sticky-header-indices [0]
110114
:data []

src/status_im/subs/wallet/wallet.cljs

+6
Original file line numberDiff line numberDiff line change
@@ -527,6 +527,12 @@
527527
(fn [ui]
528528
(get-in ui [:account-page :active-tab])))
529529

530+
(rf/reg-sub
531+
:wallet/home-tab
532+
:<- [:wallet/ui]
533+
(fn [ui]
534+
(:active-tab ui)))
535+
530536
(rf/reg-sub
531537
:wallet/aggregated-tokens
532538
:<- [:wallet/accounts-without-watched-accounts]

0 commit comments

Comments
 (0)