owncloud / android

:phone: The ownCloud Android App
GNU General Public License v2.0
3.77k stars 3.08k forks source link

[BUG] "Clear data" button malfunctioning #4350

Closed Aitorbp closed 3 months ago

Aitorbp commented 3 months ago

Related Issues

App: https://github.com/owncloud/android/issues/4309


QA

Reports:

Final behaviour:

https://github.com/owncloud/android/pull/4350#issuecomment-2036919590

jesmrec commented 3 months ago

About the removed translations, i think that removing the original string from setup.xml , transifex will remove it from all translation files... i'm not totally sure about this, any further idea?

jesmrec commented 3 months ago

(1) [FIXED]

The solution is based in a preference PREFERENCE_CLEAR_DATA_ALREADY_TRIGGERED which existence deals some actions, f. ex. removing the existent accounts. The problem comes with the migrations. The preference did not exist in advance, therefore when migration takes part, the accounts are going to be removed. That's not expected and must be fixed.

jesmrec commented 3 months ago

OTOH, it'd be useful to detail in the issue the last status update. Last comment is about suggesting the account removal, but not the final decision and why.

jesmrec commented 3 months ago

Checks performed:

Migrations:

Release mode: required change in version code, otherwise everything removed. We can assume that for release builds. Debug mode: works only for versions starting from this (containing the version code inside the shared preferences)

Go on with this. Will need another check in the release process.