-
Notifications
You must be signed in to change notification settings - Fork 987
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
deps: bump react-native-linear-gradient library #20329
Conversation
Jenkins BuildsClick to see older builds (27)
|
c55f702
to
f4ff863
Compare
f4ff863
to
9b0f036
Compare
84% of end-end tests have passed
Not executed tests (1)Failed tests (5)Click to expandClass TestGroupChatMultipleDeviceMergedNewUI:
Class TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestActivityMultipleDevicePRTwo:
Class TestWalletMultipleDevice:
Expected to fail tests (3)Click to expandClass TestWalletOneDevice:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestCommunityOneDeviceMerged:
Passed tests (43)Click to expandClass TestOneToOneChatMultipleSharedDevicesNewUi:
Class TestCommunityOneDeviceMerged:
Class TestActivityCenterContactRequestMultipleDevicePR:
Class TestCommunityMultipleDeviceMergedTwo:
Class TestGroupChatMultipleDeviceMergedNewUI:
Class TestDeepLinksOneDevice:
Class TestCommunityMultipleDeviceMerged:
Class TestActivityMultipleDevicePRTwo:
Class TestWalletOneDevice:
Class TestActivityMultipleDevicePR:
Class TestOneToOneChatMultipleSharedDevicesNewUiTwo:
|
1d9e5a8
to
c978947
Compare
c978947
to
dcc6f7e
Compare
dcc6f7e
to
1bd6365
Compare
@status-im/mobile-qa : Actually this PR just requires a design review since this library upgrade could probably have an impact on how gradients are rendered or how they look. |
@Francesca-G if you can point us to the places in the app where gradients are used, it would be really helpful!!! |
Wallet: Messages: Communities: That should be it, hopefully I'm not forgetting anything 🙈 |
Thanks @siddarthkay @Francesca-G! |
1bd6365
to
cf297cb
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.
Looks good to me 🚀
`react-native-linear-gradient` library has to be upgraded to `3.0.0-alpha.1` for new architecture support on `Android`.
cf297cb
to
0a56850
Compare
Thanks @qoqobolo and @Francesca-G for taking a look :) |
Summary
react-native-linear-gradient
library has to be upgraded to3.0.0-alpha.1
for new architecture support onAndroid
.Testing notes
Only visual elements where gradients are used would be impacted.
Platforms
status: ready
related issue : #18138