serversideup / financial-freedom

🔥🔥🔥 An open source alternative to Mint, YNAB, and more. Stay on budget and build wealth without sacrificing your privacy.
https://serversideup.net/open-source/financial-freedom/
GNU General Public License v3.0
1.67k stars 137 forks source link

Merge Add Transaction Modals #28

Closed danpastori closed 8 months ago

danpastori commented 3 years ago

Issue

Right now, we have two modals to add transactions. It'd be nice to have a single source of functionality, this way we can update and re-use the code in multiple places.

How it Happened

I was prototyping some code and needed a quick way to over-ride settings. For example, if I add a transaction on the /transactions page, the modal allows you to select the account. If I add a transaction on an individual account page, the account is pre-selected. We should allow for an account to be pre-set and the modal re-used from anywhere.

How to Solve

Merge functionality from:

Into

Need to Account For