-
Notifications
You must be signed in to change notification settings - Fork 979
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: max value is zero for some tokens #20096
Conversation
57d2158
to
f4c1f40
Compare
Jenkins BuildsClick to see older builds (21)
|
f4c1f40
to
ef1b7b0
Compare
90% of end-end tests have passed
Failed tests (3)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestWalletMultipleDevice:
Expected to fail tests (2)Click to expandClass TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (47)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletOneDevice:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
|
c13423c
to
f03aa65
Compare
Hi @briansztamfater! Could you please rebase the PR and resolve existing conflicts? Thank you! |
Yep, on it! @pavloburykh |
:layer 1}]} | ||
:wallet/current-viewing-account-tokens-filtered {:balances-per-chain {1 {:raw-balance | ||
:wallet/wallet-send-token {:symbol :eth | ||
:networks [{:source 879 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Looks like formatting needs one more run of make lint-fix
)
91f4ecb
to
dbe4788
Compare
@pavloburykh rebase done |
dbe4788
to
63ae0b0
Compare
87% of end-end tests have passed
Failed tests (5)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestWalletOneDevice:
Class TestWalletMultipleDevice:
Expected to fail tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (45)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestWalletOneDevice:
Class TestActivityMultipleDevicePR:
|
@briansztamfater thank you for the fixes. PR is ready for merge. |
Signed-off-by: Brian Sztamfater <brian@status.im>
63ae0b0
to
6ed80ba
Compare
fixes #19646
fixes #19813
fixes #19739
Summary
This PR fixes an issue where max value is 0 for some tokens and prevents it to send them
Platforms
Areas that maybe impacted
Functional
Steps to test
Case 1
Case 2
status: ready