couchbaselabs / ToDoLite-iOS

To-Do list sample app for Couchbase Lite, native iOS version
85 stars 41 forks source link

sign in with a new account isn't working #7

Closed jchris closed 10 years ago

jchris commented 10 years ago

if you already have a profile doc in the cloud, adding new devices works. Sign in for the first time fails to create a profile document. We are making deprecated calls:

        [repl setFacebookEmailAddress:userID];
        [repl registerFacebookToken:accessToken forEmailAddress:userID];

So once I fix that if it doesn't fix the issue I'll report back here.

jchris commented 10 years ago

I'm having a hard time figuring out from the docs and the Couchbase Lite iOS codebase, what exactly I should replace these calls with. They happen here: https://github.com/couchbaselabs/ToDoLite-iOS/blob/master/TodoLite7/CBLSyncManager.m#L236

Tagging @jens and @pasin -- if there is not example code I can crib from or other easy explanation maybe one of you can help me tomorrow around 11 to noon?

jens commented 10 years ago

I think you got the wrong Jens...

pasin commented 10 years ago

@jchris, how about this : repl.authenticator = [CBLAuthenticator facebookAuthenticatorWithToken:accessToken];

@snej, could you let us know if it should be otherwise?

jchris commented 10 years ago

@jens sorry about that of all people I should know better. I blame the holiday weekend.

Thanks @pasin I will try that this evening.

jchris commented 10 years ago

@pasin that seemed to work. I'm curious why I don't need to pass in the email address anymore. But it works. :) New version is on the way to the App Store.

tleyden commented 10 years ago

@jchris can you post an update once its in the app store?

Sent from my iPhone

On May 28, 2014, at 6:49 AM, Chris Anderson notifications@github.com wrote:

@pasin that seemed to work. I'm curious why I don't need to pass in the email address anymore. But it works. :) New version is on the way to the App Store.

— Reply to this email directly or view it on GitHub.

jchris commented 10 years ago

Currently Waiting for Review.

On Monday, June 2, 2014, Traun Leyden notifications@github.com wrote:

@jchris can you post an update once its in the app store?

Sent from my iPhone

On May 28, 2014, at 6:49 AM, Chris Anderson <notifications@github.com javascript:_e(%7B%7D,'cvml','notifications@github.com');> wrote:

@pasin that seemed to work. I'm curious why I don't need to pass in the email address anymore. But it works. :) New version is on the way to the App Store.

— Reply to this email directly or view it on GitHub.

— Reply to this email directly or view it on GitHub https://github.com/couchbaselabs/ToDoLite-iOS/issues/7#issuecomment-44869458 .

Chris Anderson http://jchrisa.net http://www.couchbase.com

pasin commented 10 years ago

Close the issue now.