Aptoide / aptoide-uploader

31 stars 16 forks source link

[fix/MOB-672] facebook login fail on fresh install #29

Closed jdandrade closed 4 years ago

jdandrade commented 4 years ago

What does this PR do?

Facebook login on fresh install was failing because previous to the login attempt the accessToken persisted is not null (empty string ""), which is the only validation we do. This would make it think the token was already retrieved and not retrieve a new one. We should look in the future into why is the auto-login (probable cause) saving an empty access token in the persistence layer.

Where should the reviewer start?

How should this be manually tested?

clear data to simulate fresh install > login with facebook

What are the relevant tickets?

Tickets related to this pull-request: MOB-672

Code Review Checklist