Skip to content

Cannot read property 'sendOTP' of undefined #2

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

Open
felansu opened this issue Jan 12, 2018 · 7 comments
Open

Cannot read property 'sendOTP' of undefined #2

felansu opened this issue Jan 12, 2018 · 7 comments
Labels
needs-verification some one verify & confirm for closing

Comments

@felansu
Copy link

felansu commented Jan 12, 2018

Getting error Cannot read property 'sendOTP' of undefined

@suprraz
Copy link

suprraz commented May 7, 2018

Has this been resolved? The reference above does not seem to contain relevant information.

EDIT: After digging into the iOS code, it seems the sendOTP() function has not been implemented.
Perhaps the author has the intention to complete this portion but has not done so yet. Some more clarification in the README.md would be helpful for others looking for cross-platform support.

@ashinga48
Copy link
Owner

ashinga48 commented May 8, 2018

Hey @suprraz @felansu thank you for pointing out.. dont know how it rolled back.

I have updated it, can you guys check it once and let me know.

@ashinga48 ashinga48 added the needs-verification some one verify & confirm for closing label May 8, 2018
@suprraz
Copy link

suprraz commented May 8, 2018

@ashinga48 It appears the issue is that the latest code has a version of 1.0.2 while the outdated code is versioned 1.0.3 in the npm registry. Even now, doing an npm install (tested yarn add in my case) gave me the old code. Bumping the version in package.json to 1.0.4 should fix the issue.

@ashinga48
Copy link
Owner

oops let me push it to npm even.

@ashinga48
Copy link
Owner

@suprraz can you plz check it now

@suprraz
Copy link

suprraz commented May 8, 2018

@ashinga48 Latest version (1.0.4) pulled correctly. I'm seeing another issue with the new code, in that the library is unable to find the required Firebase headers:

RNFirebasePhoneAuth.m:3:9: fatal error: 'Firebase/Firebase.h' file not found
#import <Firebase/Firebase.h>
        ^~~~~~~~~~~~~~~~~~~~~
1 error generated.

@ashinga48
Copy link
Owner

hey @suprraz let me check with a sample project.. i will upload a video on integration. Thanks. ping you in night

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
needs-verification some one verify & confirm for closing
Projects
None yet
Development

No branches or pull requests

3 participants