nazrinharris / pecunia

Massively ambitious personal finance app.
1 stars 0 forks source link

Implement a local/guest account #132

Open nazrinharris opened 10 months ago

nazrinharris commented 10 months ago

This PR will be pretty important, as it sets the foundation for multi-account operations. Even if the app technically doesn't support multiple accounts yet. I'll flesh out the details in a bit.

Here's a beginning plan. Since this'll basically be multi-account support, I'm gonna need to be able to make databases that correspond to a specific user. Easiest way to do this is to simply append the database name with the uid. For example, pecunia_0000314128312381.db. I considered updating to an encrypted database but decided to do that in the future. A list of uids where the database exists will also be kept, in the shared preferences.

So I have the saved_users map stored in shared_preferences, but I only meant that to be a record of saved users in the app. If I want to make a local account, I'm gonna need to be able to store the user's email and password hash. It's probably not a good idea to just simply store it in shared_preferences so I'll take a more secure approach.

Initial Account Deeds

Big Deed, Implement Local Authentication

Continued Deeds

Database Deeds

Enhancements

Bugs

nazrinharris commented 9 months ago

775e230 is more or less a mistake, as I tried to follow a "one-issue-one-branch-one-pr" approach, but I don't really like that. So I'll continue with what I'm doing right now, one PR, multiple issues, when it comes to features, and do the said approach for future bugs and the like.