-
Notifications
You must be signed in to change notification settings - Fork 491
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
RN .40 compatibility (tweak #import statements) #202
base: master
Are you sure you want to change the base?
Conversation
I have verified the changes and they work for me. Please merge. Note that it breaks the package for people still on 0.39 so it should be mentioned in the readme that anybody on 0.39 or earlier should use the previous version of this package. |
This works for me too. |
Don't think so. The functionality was merged into RN core, but with issues.
Bit of an awkward handoff period
…On Jan 25, 2017 6:03 PM, "Zino Hofmann" ***@***.***> wrote:
Btw is the repo still maintained?
—
You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
<#202 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AAL6gsMHZsvAzv3Re2FEcMNIybAJonU2ks5rV9SqgaJpZM4LesED>
.
|
That's putting it mildly. The RN devs are too ivory tower, so instead of something functional, it's a half finished thing that's useless to me as it breaks on 3rd party web apps. With all the PRs sitting backlogged on |
Should be merged! 👍 |
I also sent a close one: #217 |
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.
It's fine.
Please do take a careful look, I'm not an Obj-C dev. Just following the example of other projects' .39->.40 PRs. Tested in Simulator and all seems well. Compilation fails in RN .40 w/o these changes.