Git Product home page Git Product logo

Comments (14)

vincentfretin avatar vincentfretin commented on May 11, 2024 2

I guess you need at least RN 0.29 where Android permissions was introduced, it may be good to add the info in the README.

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024 1

reopened so I don't forget to update the readme

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024 1

thanks for updating the readme, @KGALLET!

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024 1

@ekryski - Hey dude, thanks for the explanation. I didn't realize that npm treated <1.0.0 and >=1.0.0 differently. Will definitely follow this rule going forward!! thanks again 🍻

from react-native-permissions.

KGALLET avatar KGALLET commented on May 11, 2024

Ok, updating to RN 0.30 solve my problem.
Thx @vincentfretin @yonahforst :)
Closed.

from react-native-permissions.

radko93 avatar radko93 commented on May 11, 2024

Getting this on RN 0.31.0, any tips?

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024

Hrmmm. I was lazy and used an undocumented API 😱 . It probably changed in 0.31.0, I'll have a look.

from react-native-permissions.

radko93 avatar radko93 commented on May 11, 2024

Probably it is my fault since my android project uses older RN version than the version installed from npm (how this is even possible?), but thanks for checking anyway :)

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024

Ok, no problem. I checked anyways and looks like the API hasn't changed. Closing the issue back out.

from react-native-permissions.

ekryski avatar ekryski commented on May 11, 2024

FWIW it probably should have been a minor release, since this has a breaking change. Anyone on a version of RN < 0.29.0 had it installed on an npm install.

from react-native-permissions.

yonahforst avatar yonahforst commented on May 11, 2024

@ekryski thanks for bringing this up. I wasn't sure if I did the version numbers right, so I really appreciate feedback here.

Android wasn't supported on previous versions so I didn't think adding it would be considered a breaking change. The old API still works but has deprecation messages. That's why I did it as a minor release (0.1.x -> 0.2.x is minor, right?). Or did you mean it should have been a major release.

Also, what do you mean here:

Anyone on a version of RN < 0.29.0 had it installed on an npm install.

Thanks for educating me on this 🙏🏽

from react-native-permissions.

ekryski avatar ekryski commented on May 11, 2024

@yonahforst no problem bud. With npm, minor releases < 1.0.0 (ie. 0.1.0) won't get automatically installed. Patch releases will (ie. 0.1.1).

Above 1.0.0 only major releases won't be installed automatically, minor and patch releases will.

So if you are < 1.0.0 (which is totally fine especially in RN because shit is changing so much) then just do a minor release for breaking changes. With feathers we don't bump to a 1.0 until we feel like it is "stable". A bit wishy-washy but it's been working well for us so far.

Android wasn't supported on previous versions so I didn't think adding it would be considered a breaking change.

I totally understand. How the hell are you supposed to know? Definitely no blame here. The issue was that doing an npm install, brought in references to Facebook's permissions in the Android gradle files, and unless you are v0.29+ of RN it just blows up compiling. So you wouldn't see it if you are only running RN on an iOS device.

Hope that explains things! Really appreciate the hard work! 😄

from react-native-permissions.

perusopersonale avatar perusopersonale commented on May 11, 2024

I'm having this problem too.
npm list say that I'm using RN v0.32.1

from react-native-permissions.

xwartz avatar xwartz commented on May 11, 2024

I have the same issue, RN v0.36.

from react-native-permissions.

Related Issues (20)

Recommend Projects

  • React photo React

    A declarative, efficient, and flexible JavaScript library for building user interfaces.

  • Vue.js photo Vue.js

    🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.

  • Typescript photo Typescript

    TypeScript is a superset of JavaScript that compiles to clean JavaScript output.

  • TensorFlow photo TensorFlow

    An Open Source Machine Learning Framework for Everyone

  • Django photo Django

    The Web framework for perfectionists with deadlines.

  • D3 photo D3

    Bring data to life with SVG, Canvas and HTML. 📊📈🎉

Recommend Topics

  • javascript

    JavaScript (JS) is a lightweight interpreted programming language with first-class functions.

  • web

    Some thing interesting about web. New door for the world.

  • server

    A server is a program made to process requests and deliver data to clients.

  • Machine learning

    Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.

  • Game

    Some thing interesting about game, make everyone happy.

Recommend Org

  • Facebook photo Facebook

    We are working to build community through open source technology. NB: members must have two-factor auth.

  • Microsoft photo Microsoft

    Open source projects and samples from Microsoft.

  • Google photo Google

    Google ❤️ Open Source for everyone.

  • D3 photo D3

    Data-Driven Documents codes.