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

fix cursor check to null #13

Merged
merged 1 commit into from
Nov 6, 2015
Merged

fix cursor check to null #13

merged 1 commit into from
Nov 6, 2015

Conversation

nucleartux
Copy link
Contributor

fixes #10

rt2zz added a commit that referenced this pull request Nov 6, 2015
@rt2zz rt2zz merged commit 3934576 into morenoh149:master Nov 6, 2015
@rt2zz
Copy link
Collaborator

rt2zz commented Nov 6, 2015

👍 thank you!

@niftylettuce
Copy link

Can we cut another release for this? I'm testing it out today.

@niftylettuce
Copy link

@nucleartux @rt2zz still getting errors, here's a screenshot...

screenshot_2015-11-21-21-27-50

I'm calling getAll and this happens - I think you need to do some other stuff than just this null fix to get it to work properly. Maybe take a look at how PhoneGap/Cordova wrote their plugin? I will gladly pay if you get this working @nucleartux

@nucleartux
Copy link
Contributor Author

Something wrong with your code(reactNativeContacts2?). Can you copy paste it here ?

@niftylettuce
Copy link

I did npm install --save rt2zz/react-native-contacts, so the code is the
same as master branch here on GitHub.
On Nov 22, 2015 1:30 AM, "Igor" notifications@github.com wrote:

Something wrong with your code(reactNativeContacts2?). Can you copy paste
it here ?


Reply to this email directly or view it on GitHub
#13 (comment)
.

@nucleartux
Copy link
Contributor Author

Please try
direct call NativeModules.Contacts.getAll

@niftylettuce
Copy link

I get a lot of this console output:

screen shot 2015-11-22 at 2 09 28 am

@rt2zz
Copy link
Collaborator

rt2zz commented Nov 22, 2015

I do not have an answer at the moment, but from a bigger picture the current code (which I wrote) is woefully slow, and at the very least would need to be pushed to a background thread before it could be considered usable.

I do not have a timetable for fixing this as of now, but am open to suggestions / PRs.

@niftylettuce if you are interested in paying I can probably get you in touch with someone who can hammer this out.

@niftylettuce
Copy link

please do put me in touch, its way too slow. niftylettuce@gmail.com
On Nov 22, 2015 2:26 AM, "Zack" notifications@github.com wrote:

I do not have an answer at the moment, but from a bigger picture the
current code (which I wrote) is woefully slow, and at the very least would
need to be pushed to a background thread before it could be considered
usable.

I do not have a timetable for fixing this as of now, but am open to
suggestions / PRs.

@niftylettuce https://github.com/niftylettuce if you are interested in
paying I can probably get you in touch with someone who can hammer this out.


Reply to this email directly or view it on GitHub
#13 (comment)
.

@niftylettuce
Copy link

@nucleartux it runs, but it's super slow, and some of the contacts come back blank -- like the nulls are never cleaned up or something? It's just super slow in general - how can we make this fast? /cc @rt2zz

@rt2zz rt2zz mentioned this pull request Dec 16, 2015
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.

Android getAllContacts not working
3 participants