Aptoide / aptoide-uploader

31 stars 16 forks source link

[APP-744] Fix update while logged in #47

Closed phillipsgoncalves closed 2 years ago

phillipsgoncalves commented 2 years ago

What does this PR do?

This PR aims at fixing some errors we had when updating the uploader app. It had several problems: database migration and then we noticed that it had 2 more issues:

First one was the the avatar was null as in the last version (where the login was stored) we didnt need the avatar, so it was not being saved.

Second one was that on an upgrade from one version to the other, since the user was already logged in, the start of the installed apps view would be faster and the database was not populated yet, showing 0 apps.

Where should the reviewer start?

How should this be manually tested?

Migrate from a previous version to this one and check that everything is ok. Not only DB migration, but also the issues mentioned above

What are the relevant tickets?

Tickets related to this pull-request: APP-744

Questions:

Does this add new dependencies which need to be added? (Eg. new keys, etc.)

Code Review Checklist