-
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
chore(wallet): clean up root folder #19417
Conversation
@@ -403,9 +402,6 @@ | |||
:options options/transparent-screen-options | |||
:component wallet-share-address/view} | |||
|
|||
{:name :screen/wallet.saved-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.
this screen is unused since we made the initial skeleton for wallet.
@@ -58,11 +58,14 @@ | |||
[status-im.contexts.wallet.account.bridge-send.view :as wallet-bridge-send] | |||
[status-im.contexts.wallet.account.bridge-to.view :as wallet-bridge-to] | |||
[status-im.contexts.wallet.account.bridge.view :as wallet-bridge] |
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.
just moved these ns around to clean up the wallet root folder somewhat
Jenkins BuildsClick to see older builds (4)
|
I believe this can skip manual QA as it is just moving some folders around. |
907ac36
to
ffddc2a
Compare
94% of end-end tests have passed
Failed tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (45)Click to expandClass TestActivityMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
@status-im/mobile-qa - the e2e results look okay? |
Hi @J-Son89 ! Let me re-check 2 failed ones. |
Hey @J-Son89! E2E are okay. Failures are not PR related. |
ffddc2a
to
a3c83c6
Compare
100% of end-end tests have passed
Passed tests (2)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityMultipleDeviceMergedTwo:
|
This pr moves some namespaces in the wallet root folder into some more appropriate sub-folders to keep the codebase somewhat more organised at the top level.