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

Do not crash iOS Simulator #151

Merged

Conversation

swissmanu
Copy link
Contributor

@swissmanu swissmanu commented Aug 30, 2018

This ports changes suggested by @barbsicle to react-native-sensors latest version.

Original Pull Request: #132

Changes included do not touch any JS implementation. It's iOS/Objective-C only.

This ports changes suggested by @barbsicle to react-native-sensors
latest version.

Original Pull Request:
react-native-sensors#132
bra-shelker added a commit to bra-shelker/react-native-sensors that referenced this pull request Sep 13, 2018
@miracle2k
Copy link
Contributor

I just created the same patch. Can we get this merged, please?

@DanielMSchmidt
Copy link
Member

Merging won't help, I am currently unable to deploy due to conditions on my machine :( plus I could not get semantic release to work to have this automated

@barbsicle
Copy link

You may also want to implement the quick observable fix in src/decorator.js, as it would still crash on simulator without it (once the component unmounts).

Also, @swissmanu @miracle2k and whoever else is waiting on this, I have been running on my fork for quite awhile now, and I suggest that as a temporary workaround so you don't have to keep editing node_modules.

@DanielMSchmidt DanielMSchmidt merged commit 2165c75 into react-native-sensors:master Sep 17, 2018
@DanielMSchmidt
Copy link
Member

🎉 This PR is included in version 4.0.0 🎉

The release is available on:

Your semantic-release bot 📦🚀

kibolho pushed a commit to kibolho/react-native-sensors that referenced this pull request Oct 7, 2023
This ports changes suggested by @barbsicle to react-native-sensors
latest version.

Original Pull Request:
react-native-sensors#132

BREAKING CHANGE: we change that we reject without an error now, which might change the usage for some folks
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

4 participants