Clear entire TinyDB when switching accounts. #578

New Issue

Closed

opened 2020-07-06 23:47:06 +00:00 by opyale · 4 comments

opyale commented

2020-07-06 23:47:06 +00:00

Collaborator

This is important so that old and left over values do not possibly affect the functioning of another account.

👍 1

opyale added the

🌠 Long-term

🔒 Security

labels 2020-07-06 23:47:06 +00:00

opyale commented

2020-07-06 23:52:55 +00:00

Poster

Collaborator

A method for this is already implemented in TinyDB.

/**
 * Clear SharedPreferences (remove everything)
 */
public void clear() {
    preferences.edit().clear().apply();
}

It only needs to be called in the logout() method.

A method for this is already implemented in ``TinyDB``. ``` /** * Clear SharedPreferences (remove everything) */ public void clear() { preferences.edit().clear().apply(); } ``` It only needs to be called in the ``logout()`` method.

~~opyale referenced this issue 2020-07-06 23:54:53 +00:00~~

Accounts overview screen #577

mmarif commented

2020-07-07 06:22:20 +00:00

Owner

We cant at this moment use this until all the data(e.g settings) are migrated to db for example.

Bdw what data would effect the functionality of the app in tinydb if remain there?

We cant at this moment use this until all the data(e.g settings) are migrated to db for example. Bdw what data would effect the functionality of the app in tinydb if remain there?

mmarif commented

2020-07-07 06:25:33 +00:00

Owner

And I do use the app(testing) with multiple instances and multiple accounts and never came across a single issue so far except a token bug(came across recently) which will be handeled later with multiple accounts setup.

opyale added a new dependency 2021-04-07 22:03:53 +00:00

#876 Move preferences from TinyDB to database

opyale added this to the Database-related work project 2021-06-17 17:28:21 +00:00