search

powerpaul17 / nc_money

Nextcloud app for finance management
GNU Affero General Public License v3.0
97 stars 9 forks source link

[Feature Request] Remove Accounts #10

Closed meichthys closed 1 year ago

meichthys commented 1 year ago

Currently there are 11 Accounts when first opening the app, but there doesn't seem to be a way to delete accounts. image

powerpaul17 commented 1 year ago

Yes, you are right, I have not implemented deleting accounts. The problem is that there can be a lot of data loss if you for example delete an account by accident so I am still thinking about the best approach here..

meichthys commented 1 year ago

Ok perhaps the 11 'initial' accounts I was seeing were accounts that I had setup when trying this app in the past.

A simple deletion confirmation dialog with a scary warning would probably suffice for now.

csolisr commented 1 year ago

Perhaps to ease the data loss part, there should also be an option to move data between accounts of the same type, and a warning if a particular account is not yet empty.

powerpaul17 commented 1 year ago

Implemented in 666bd379fdcaaf1039a48deaed80e11ea6b64981