Skip to content
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

Use react-native-community CameraRoll rather than the one from core react-native #93

Closed
wants to merge 2 commits into from

Conversation

nshaposhnik
Copy link

@nshaposhnik nshaposhnik commented Jul 2, 2019

I've encountered an issue with react-native 0.59.9, when I use groupTypes='All', it's showing some duplicates in iOS. I've tried to use react-native-community/cameraroll (which is created as a part of lean core project), and now it's all working correctly. In any case it should be updated sooner or later to use this dependency, as the react-native core package will be no longer updated from some point (and it lags behind already).

@nshaposhnik nshaposhnik changed the title Use lean core CameraRoll rather than the one from react-native Use react-native-community CameraRoll rather than the one from core react-native Jul 2, 2019
@marbetschar
Copy link

Would love to see this pull request merged. It finally fixed all the quirks I was experiencing using the react-native cameraroll implementation.

Please update the dependency to react-native-community/cameraroll - it took me hours to figure out what exactly is wrong. And it seems there are others dealing with the same issue:

@@ -127,6 +127,7 @@ android {

dependencies {
compile fileTree(dir: "libs", include: ["*.jar"])
compile project(':@react-native-community_cameraroll')
Copy link
Collaborator

Choose a reason for hiding this comment

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

autolink should manage this

@@ -7,6 +7,7 @@
"url": "https://github.com/jeanpan/react-native-camera-roll-picker/issues"
},
"dependencies": {
"@react-native-community/cameraroll": "^1.1.1",
Copy link
Collaborator

Choose a reason for hiding this comment

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

this should be a peer dep

@sibelius
Copy link
Collaborator

fixed here #102

tks

@sibelius sibelius closed this Nov 19, 2019
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.

None yet

3 participants