Skip to content

Conversation

@Definitely-Not-Vlad
Copy link
Contributor

Updated dependencies for forks and out-of-date packages.

Changed Image component to be compatible with RN 0.50.0 ImageBackground component.

Removed obsolete prop resizeMode from theme.js.

@Definitely-Not-Vlad Definitely-Not-Vlad self-assigned this Dec 4, 2017
@sky-franciscogoncalves
Copy link

sky-franciscogoncalves commented Dec 7, 2017

Hi @Vladimir-Vdovic,

I tried to add your PR to my project and pointed both themes and animations dependencies to your rn-50-update PRs. However, I was not able to get my app running as PropTypes error was presented.

Do you have any recommendation on how to get this working?

@Definitely-Not-Vlad
Copy link
Contributor Author

Hey @sky-franciscogoncalves,

Just use npm i shoutem/ui#feature/rn-50-update --save and you should be good to go. Theme and Animation have been released.

color: bufferColor
}];

if(!NativeModules.BVLinearGradient){
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Put it on top of the function.

package.json Outdated
"lodash": "~4.15.0",
"qs": "~4.0.0",
"react-native-lightbox": "shoutem/react-native-lightbox#4e58b9b1aca166142d69b161ff511efc3358ff9f",
"react-native-lightbox": "shoutem/react-native-lightbox#master",
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Don't point to master directly, add a tag to both repos and point to tag. This might get broken if someone commits to master.

@Definitely-Not-Vlad Definitely-Not-Vlad merged commit bc9a4aa into develop Dec 8, 2017
@Definitely-Not-Vlad Definitely-Not-Vlad deleted the feature/rn-50-update branch December 18, 2017 11:46
@JulianKingman
Copy link

JulianKingman commented Jan 8, 2018

@Vladimir-Vdovic I'm getting resizeMode warnings, is that related to this PR? How to fix?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants