-
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
Move selected addresses for permissions to app db #18430
Move selected addresses for permissions to app db #18430
Conversation
Jenkins BuildsClick to see older builds (20)
|
d232d0e
to
09498c5
Compare
09498c5
to
6669baa
Compare
:communities/previous-permission-addresses addresses | ||
:communities/selected-permission-addresses addresses)})) |
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.
:communities/selected-permission-addresses
- This key will track the selected address for both passing it to the status-go API and displaying it in the UI.
:communities/previous-permission-addresses
- This key is specifically for storing the previously selected state. It enables us to reset to the previous address when the user clicks 'Cancel' after making an edit.
@ilmotta, I hope these new key names make more sense.
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.
It's perfect @ajayesivan. You even managed to come up with names with the same length, god mode.
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.
It was a pain 😓 I renamed it 4 or 5 times and finally ended up with this.
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.
d7f77a5
to
3c1b88f
Compare
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.
Nicely done!
:communities/previous-permission-addresses addresses | ||
:communities/selected-permission-addresses addresses)})) |
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.
It's perfect @ajayesivan. You even managed to come up with names with the same length, god mode.
src/status_im/subs/root.cljs
Outdated
@@ -145,7 +145,8 @@ | |||
(reg-root-key-sub :communities/collapsed-categories :communities/collapsed-categories) | |||
(reg-root-key-sub :communities/selected-tab :communities/selected-tab) | |||
(reg-root-key-sub :contract-communities :contract-communities) | |||
(reg-root-key-sub :communities/addresses-for-permissions :communities/addresses-for-permissions) | |||
(reg-root-key-sub :communities/selected-permission-addresses |
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.
The args fit on the same line.
3c1b88f
to
bd81420
Compare
bd81420
to
6dc2d71
Compare
4% of end-end tests have passed
Failed tests (45)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (2)Click to expandClass TestDeepLinksOneDevice:
|
79% of end-end tests have passed
Failed tests (8)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePRTwo:
Expected to fail tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (38)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
|
6dc2d71
to
62707b9
Compare
fixes #18405
status: ready