FirebaseExtended / make-it-so-android

Apache License 2.0
214 stars 58 forks source link

Remove unnecessary userId update #18

Closed marinacoelho closed 1 year ago

marinacoelho commented 1 year ago

Now that the linkAccount() method is being used correctly, we don't need to update the userId anymore, because it remains the same.

mtali commented 1 year ago

@marinacoelho

I think updating userId depends on the scenario (User flows)

Anonymous User -> Create Account Well, because we linking the accounts, userId will be preserved so no need to update userId

Anonymous User -> Sign in to existing account Here we might need to update the tasks created with an anonymous account to the signed-in account because we don't link user accounts during sign in the process only authenticating the user