-
Notifications
You must be signed in to change notification settings - Fork 986
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] Remove POKT Sepolia and Goerli RPC URL #19040
Conversation
Jenkins BuildsClick to see older builds (16)
|
43b29da
to
7b3b4bf
Compare
7b3b4bf
to
e633e43
Compare
8% of end-end tests have passed
Failed tests (43)Click to expandClass TestCommunityMultipleDeviceMergedTwo:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (4)Click to expandClass TestCommunityOneDeviceMerged:
|
17% of end-end tests have passed
Failed tests (39)Click to expandClass TestCommunityMultipleDeviceMerged:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePR:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityOneDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMergedTwo:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (8)Click to expandClass TestActivityCenterContactRequestMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityOneDeviceMerged:
|
hi @smohamedjavid could you please rebase current and status go PR? |
e633e43
to
f64d65e
Compare
f64d65e
to
5a09d22
Compare
@VolodLytvynenko - Done. Ready for testing! Please note that this PR has been updated to use Infura for Sepolia as well, as POKT dropped support. |
96% of end-end tests have passed
Failed tests (1)Click to expandClass TestCommunityMultipleDeviceMerged:
Expected to fail tests (1)Click to expandClass TestCommunityOneDeviceMerged:
Passed tests (46)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestActivityMultipleDevicePR:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
Class TestCommunityMultipleDeviceMerged:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
|
Hi @smohamedjavid thank you for PR. No issues from my side. PR is ready to be merged |
status-im/status-go@00f50a4...bda33d7 Signed-off-by: Mohamed Javid <[email protected]>
status-im/status-go@00f50a4...3a06c0e Signed-off-by: Mohamed Javid <[email protected]>
status-im/status-go@00f50a4...9e1d65b Signed-off-by: Mohamed Javid <[email protected]>
5a09d22
to
37c36e7
Compare
Summary
This PR removes the POKT (Grove) URL for Sepolia and Goerli as POKT dropped support.
Now, we will use Infura RPC for Sepolia and Goerli as default.
status-go
PR: status-im/status-go#4830Testing notes
A quick test on the wallet with the Sepolia and Goerli test network is appreciated.
Platforms
status: ready