fix: no routes found error when disabling chains and current amount exceeds new max amount

This commit is contained in:
Brian Sztamfater 2024-05-20 19:48:15 +02:00
parent 389a730eff
commit a819c50958
No known key found for this signature in database
GPG Key ID: 59EB921E0706B48F
4 changed files with 51 additions and 17 deletions

View File

@ -64,7 +64,7 @@
:token-networks-ids token-networks-ids :token-networks-ids token-networks-ids
:tx-type tx-type :tx-type tx-type
:receiver? false}) :receiver? false})
(send-utils/reset-network-amounts-to-zero (send-utils/reset-loading-network-amounts-to-zero
sender-network-values)) sender-network-values))
receiver-network-values (if routes-available? receiver-network-values (if routes-available?
(send-utils/network-amounts (send-utils/network-amounts
@ -74,7 +74,7 @@
:token-networks-ids token-networks-ids :token-networks-ids token-networks-ids
:tx-type tx-type :tx-type tx-type
:receiver? true}) :receiver? true})
(send-utils/reset-network-amounts-to-zero (send-utils/reset-loading-network-amounts-to-zero
receiver-network-values)) receiver-network-values))
network-links (when routes-available? network-links (when routes-available?
(send-utils/network-links chosen-route (send-utils/network-links chosen-route
@ -93,9 +93,9 @@
(rf/reg-event-fx :wallet/suggested-routes-error (rf/reg-event-fx :wallet/suggested-routes-error
(fn [{:keys [db]} [error]] (fn [{:keys [db]} [error]]
(let [cleaned-sender-network-values (-> (get-in db [:wallet :ui :send :sender-network-values]) (let [cleaned-sender-network-values (-> (get-in db [:wallet :ui :send :sender-network-values])
(send-utils/reset-network-amounts-to-zero)) (send-utils/reset-loading-network-amounts-to-zero))
cleaned-receiver-network-values (-> (get-in db [:wallet :ui :send :receiver-network-values]) cleaned-receiver-network-values (-> (get-in db [:wallet :ui :send :receiver-network-values])
(send-utils/reset-network-amounts-to-zero))] (send-utils/reset-loading-network-amounts-to-zero))]
{:db (-> db {:db (-> db
(update-in [:wallet :ui :send] (update-in [:wallet :ui :send]
dissoc dissoc
@ -299,9 +299,27 @@
(rf/reg-event-fx :wallet/disable-from-networks (rf/reg-event-fx :wallet/disable-from-networks
(fn [{:keys [db]} [chain-ids]] (fn [{:keys [db]} [chain-ids]]
{:db (-> db {:db (assoc-in db [:wallet :ui :send :disabled-from-chain-ids] chain-ids)}))
(assoc-in [:wallet :ui :send :disabled-from-chain-ids] chain-ids)
(assoc-in [:wallet :ui :send :loading-suggested-routes?] true))})) (rf/reg-event-fx :wallet/reset-network-amounts-to-zero
(fn [{:keys [db]}]
(let [sender-network-values (get-in db [:wallet :ui :send :sender-network-values])
receiver-network-values (get-in db [:wallet :ui :send :receiver-network-values])
disabled-from-chain-ids (get-in db [:wallet :ui :send :disabled-from-chain-ids])
sender-network-values (send-utils/reset-network-amounts-to-zero
{:network-amounts sender-network-values
:disabled-chain-ids disabled-from-chain-ids})
receiver-network-values (send-utils/reset-network-amounts-to-zero
{:network-amounts receiver-network-values
:disabled-chain-ids []})]
{:db (-> db
(assoc-in [:wallet :ui :send :sender-network-values] sender-network-values)
(assoc-in [:wallet :ui :send :receiver-network-values] receiver-network-values)
(update-in [:wallet :ui :send]
dissoc
:network-links
(when (empty? sender-network-values) :sender-network-values)
(when (empty? receiver-network-values) :receiver-network-values)))})))
(rf/reg-event-fx :wallet/get-suggested-routes (rf/reg-event-fx :wallet/get-suggested-routes
(fn [{:keys [db now]} [{:keys [amount]}]] (fn [{:keys [db now]} [{:keys [amount]}]]

View File

@ -276,7 +276,9 @@
(set-input-state #(controlled-input/set-upper-limit % current-limit))) (set-input-state #(controlled-input/set-upper-limit % current-limit)))
[current-limit]) [current-limit])
(rn/use-effect (rn/use-effect
#(when input-error (debounce/clear-all)) (fn []
(when input-error (debounce/clear-all))
(when routes (rf/dispatch [:wallet/reset-network-amounts-to-zero])))
[input-error]) [input-error])
[rn/view [rn/view
{:style style/screen {:style style/screen
@ -328,13 +330,13 @@
sender-network-values sender-network-values
token-not-supported-in-receiver-networks?) token-not-supported-in-receiver-networks?)
[token-not-available token-symbol receiver-networks token-networks]) [token-not-available token-symbol receiver-networks token-networks])
(when (or loading-routes? (seq route)) (when (or loading-routes? fee-formatted)
[estimated-fees [estimated-fees
{:loading-routes? loading-routes? {:loading-routes? loading-routes?
:fees fee-formatted :fees fee-formatted
:amount amount-text :amount amount-text
:receiver (address/get-shortened-key to-address)}]) :receiver (address/get-shortened-key to-address)}])
(when (or no-routes-found? limit-insufficient?) (when (and (or no-routes-found? limit-insufficient?) (not-empty sender-network-values))
[rn/view {:style style/no-routes-found-container} [rn/view {:style style/no-routes-found-container}
[quo/info-message [quo/info-message
{:type :error {:type :error

View File

@ -83,7 +83,7 @@
(def ^:private available-networks-count (def ^:private available-networks-count
(count (set (keys network-priority-score)))) (count (set (keys network-priority-score))))
(defn reset-network-amounts-to-zero (defn reset-loading-network-amounts-to-zero
[network-amounts] [network-amounts]
(map (map
(fn [network-amount] (fn [network-amount]
@ -93,6 +93,20 @@
:type :default))) :type :default)))
network-amounts)) network-amounts))
(defn reset-network-amounts-to-zero
[{:keys [network-amounts disabled-chain-ids]}]
(map
(fn [network-amount]
(let [disabled-chain-ids-set (set disabled-chain-ids)
disabled? (contains? disabled-chain-ids-set
(:chain-id network-amount))]
(cond-> network-amount
(and (not= (:type network-amount) :add)
(not= (:type network-amount) :not-available))
(assoc :total-amount (money/bignumber "0")
:type (if disabled? :disabled :default)))))
network-amounts))
(defn network-amounts (defn network-amounts
[{:keys [network-values disabled-chain-ids receiver-networks token-networks-ids tx-type receiver?]}] [{:keys [network-values disabled-chain-ids receiver-networks token-networks-ids tx-type receiver?]}]
(let [disabled-set (set disabled-chain-ids) (let [disabled-set (set disabled-chain-ids)

View File

@ -209,13 +209,13 @@
:disabled-chain-ids :disabled-chain-ids
disabled-chain-ids})))))) disabled-chain-ids}))))))
(deftest test-reset-network-amounts-to-zero (deftest test-reset-loading-network-amounts-to-zero
(testing "Correctly resets loading network amounts to zero and changes type to default" (testing "Correctly resets loading network amounts to zero and changes type to default"
(let [network-amounts [{:chain-id 1 :total-amount (money/bignumber "100") :type :loading} (let [network-amounts [{:chain-id 1 :total-amount (money/bignumber "100") :type :loading}
{:chain-id 10 :total-amount (money/bignumber "200") :type :default}] {:chain-id 10 :total-amount (money/bignumber "200") :type :default}]
expected [{:chain-id 1 :total-amount (money/bignumber "0") :type :default} expected [{:chain-id 1 :total-amount (money/bignumber "0") :type :default}
{:chain-id 10 :total-amount (money/bignumber "200") :type :default}] {:chain-id 10 :total-amount (money/bignumber "200") :type :default}]
result (utils/reset-network-amounts-to-zero network-amounts) result (utils/reset-loading-network-amounts-to-zero network-amounts)
comparisons (map #(map/deep-compare %1 %2) comparisons (map #(map/deep-compare %1 %2)
expected expected
result)] result)]
@ -226,7 +226,7 @@
{:chain-id 10 :total-amount (money/bignumber "0") :type :disabled}] {:chain-id 10 :total-amount (money/bignumber "0") :type :disabled}]
expected [{:chain-id 1 :total-amount (money/bignumber "100") :type :default} expected [{:chain-id 1 :total-amount (money/bignumber "100") :type :default}
{:chain-id 10 :total-amount (money/bignumber "0") :type :disabled}] {:chain-id 10 :total-amount (money/bignumber "0") :type :disabled}]
result (utils/reset-network-amounts-to-zero network-amounts) result (utils/reset-loading-network-amounts-to-zero network-amounts)
comparisons (map #(map/deep-compare %1 %2) comparisons (map #(map/deep-compare %1 %2)
expected expected
result)] result)]
@ -235,7 +235,7 @@
(testing "Processes an empty list without error" (testing "Processes an empty list without error"
(let [network-amounts [] (let [network-amounts []
expected [] expected []
result (utils/reset-network-amounts-to-zero network-amounts) result (utils/reset-loading-network-amounts-to-zero network-amounts)
comparisons (map #(map/deep-compare %1 %2) comparisons (map #(map/deep-compare %1 %2)
expected expected
result)] result)]
@ -246,7 +246,7 @@
{:chain-id 10 :total-amount (money/bignumber "200") :type :loading}] {:chain-id 10 :total-amount (money/bignumber "200") :type :loading}]
expected [{:chain-id 1 :total-amount (money/bignumber "0") :type :default} expected [{:chain-id 1 :total-amount (money/bignumber "0") :type :default}
{:chain-id 10 :total-amount (money/bignumber "0") :type :default}] {:chain-id 10 :total-amount (money/bignumber "0") :type :default}]
result (utils/reset-network-amounts-to-zero network-amounts) result (utils/reset-loading-network-amounts-to-zero network-amounts)
comparisons (map #(map/deep-compare %1 %2) comparisons (map #(map/deep-compare %1 %2)
expected expected
result)] result)]
@ -261,7 +261,7 @@
{:chain-id 10 :total-amount (money/bignumber "200") :type :default} {:chain-id 10 :total-amount (money/bignumber "200") :type :default}
{:chain-id 42161 :total-amount (money/bignumber "0") :type :default} {:chain-id 42161 :total-amount (money/bignumber "0") :type :default}
{:chain-id 59144 :total-amount (money/bignumber "0") :type :disabled}] {:chain-id 59144 :total-amount (money/bignumber "0") :type :disabled}]
result (utils/reset-network-amounts-to-zero network-amounts) result (utils/reset-loading-network-amounts-to-zero network-amounts)
comparisons (map #(map/deep-compare %1 %2) comparisons (map #(map/deep-compare %1 %2)
expected expected
result)] result)]