-
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
[#18752] fix: The W/O address shouldn't be in account selection #18809
Conversation
Jenkins BuildsClick to see older builds (12)
|
38% of end-end tests have passed
Failed tests (29)Click to expandClass TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (18)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
76% of end-end tests have passed
Failed tests (7)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestGroupChatMultipleDeviceMergedNewUI:
Passed tests (22)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
Thanks for the fix @mohsen-ghafouri! ISSUE 1: Watch-only accounts are still visible on the
|
@mohsen-ghafouri also there is still a problem that the watch-only account balance also counts when checking if you meet requirements. That is, my account has 0 balance. As you can see in the video, I can send a request to join the community, although the video_2024-02-14_16-09-33.mp4 |
@qoqobolo could you please check again for issue 1? second issue doesn't seems to be related to this PR and looks more complicated |
Thank you @mohsen-ghafouri, Issue 1 is fixed. |
@qoqobolo I just tested my PR with @cammellos changes in status-go and now it looks fine. could you please check again the second issue? |
Thanks @mohsen-ghafouri and @cammellos , great job! |
45b9e98
to
9e3924e
Compare
fixes #18752
Summary
We should not include watch-only addresses in account selection when joining a community
Result
Simulator.Screen.Recording.-.iPhone.13.-.2024-02-13.at.23.11.35.mp4
status: ready