-
Notifications
You must be signed in to change notification settings - Fork 984
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
[Feature] Show "Testnet mode enabled" banner if testnet enabled #19348
[Feature] Show "Testnet mode enabled" banner if testnet enabled #19348
Conversation
Jenkins BuildsClick to see older builds (24)
|
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.
Hey @smohamedjavid
the code looks good 👍
but I think you are missing the issue being fixed, also a link to designs would help
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.
finally, lets see alert banner in action 🔥
Added @ulisesmac 🙏 |
Yes 🔥 Thanks to you for the banner implementation 😄 |
7097eee
to
1e298b5
Compare
79% of end-end tests have passed
Failed tests (9)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (38)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
Hi @smohamedjavid ! Can you please resolve conflicts and rebase your PR? |
1e298b5
to
41cdb63
Compare
@mariia-skrypnyk - Ready for testing 👍 |
41cdb63
to
ab03040
Compare
Hi @smohamedjavid ! I am facing several issues: ISSUE 1: can not open User's Profile, it is not clickable
Actual result: user can not open his Profile Expected result: Profile opens by tap on avatar ISSUE 2: Some screen names are overlapped
Expected result: |
Hi @mariia-skrypnyk - Thanks for testing this PR.
I tried out the latest PR build on both Android and iOS, It's working fine. 🤔
I believe this is highlighted in the Alert banner Implementation PR #19011 (comment). This is expected as these are old screens and will be removed and rebuilt later. NOTE: This PR adds the banner when testnet mode is enabled and doesn't make any other changes to the component or other features in the application. I found one other issue. The testnet banner is shown on the onboarding flow after the keys are generated on profile creation.
I will fix this 👍 |
ab03040
to
0dc7a6d
Compare
This is fixed now. |
Hi @smohamedjavid ! Yes. It is a tricky issue. Can you resolve conflicts and rebase? |
0dc7a6d
to
5ffb139
Compare
Sure, thanks! 🙏
Done 👍 |
@smohamedjavid thanks for rebase. Yes, I didn't catch this profile untapable issue but I catched thin red line above profile and above banner at the top of the app when I reopen app from the background. IMG_7835.MP4Can it be fixed in this PR? |
@mariia-skrypnyk |
Yes @smohamedjavid it is really barely noticeable. Will do a separate issue. |
0% of end-end tests have passed
Failed tests (9)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMerged:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
|
@smohamedjavid you can merge your PR, failed e2e will need a fix from our side. |
hi @smohamedjavid , I've just pushed a fix for e2e tests, please don't remove my commit when rebasing |
Signed-off-by: Mohamed Javid <19339952+smohamedjavid@users.noreply.github.com>
23a4ec6
to
07b10cf
Compare
56% of end-end tests have passed
Failed tests (4)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityMultipleDeviceMerged:
Passed tests (5)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
|
fixes #19357
Summary
This PR adds the "Testnet mode enabled" banner if the User has enabled the testnet.
Platforms
Steps to test
Profile > Advanced
Testnet mode
toggleSince this commit, 1d44472 is merged. This enables Test Networks in Dev, E2E and PR Builds by default and disables in Release/Nightly Builds.
This enables in create account or restore account flow. If you upgrade the app from the old build with an existing profile, it will use the last saved (test networks enabled) state of that profile.
status: ready