removed bugged then and fail method on signIn method and switched registering 2 callback signInSuccess and signInError to handle the two different cases