-
Notifications
You must be signed in to change notification settings - Fork 985
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: error is shown when asset is selected on select assets screen #18326
Conversation
Jenkins BuildsClick to see older builds (8)
|
2e684b3
to
832b6ef
Compare
85% of end-end tests have passed
Failed tests (3)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Expected to fail tests (4)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Passed tests (41)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestDeepLinksOneDevice:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
|
832b6ef
to
4047e68
Compare
0% of end-end tests have passed
Failed tests (1)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
|
Hi @briansztamfater ! Thanks for your PR. Yes,hopefully the error is not shown anymore :) Please, merge your fix and rebase this PR |
Signed-off-by: Brian Sztamfater <brian@status.im>
4047e68
to
b8d2694
Compare
fixes #18228
Summary
Based on the logs provided I can guess this line
status-mobile/src/status_im/contexts/wallet/send/input_amount/view.cljs
Line 19 in 2e684b3
Relevant logs:
Platforms
Functional
Steps to test
status: ready