feat: Multi-account financial overview dashboard (bounty #132)#940
Open
alexanderxfgl-bit wants to merge 1 commit intorohitdash08:mainfrom
Open
feat: Multi-account financial overview dashboard (bounty #132)#940alexanderxfgl-bit wants to merge 1 commit intorohitdash08:mainfrom
alexanderxfgl-bit wants to merge 1 commit intorohitdash08:mainfrom
Conversation
) Implements a multi-account system for viewing all financial accounts in one unified dashboard with per-type breakdowns and transactions. - FinancialAccount model: name, type (checking/savings/credit_card/investment/cash), institution, balance, currency, soft-delete via is_active - AccountTransaction model: per-account transactions with auto balance update - AccountService: CRUD, overview aggregation, transaction management - REST API: POST/GET/PATCH/DELETE /accounts, GET /accounts/overview, POST/GET /accounts/<id>/transactions - PostgreSQL schema with account_type enum and optimized indexes - 11 tests covering CRUD, overview, transactions, validation Closes rohitdash08#132
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Bounty Claim: $200 — Issue #132
Implementation
Full multi-account financial overview system with account types, per-type breakdowns, and transaction tracking.
Backend Changes
Models (
app/models.py):AccountTypeenum: checking, savings, credit_card, investment, cashFinancialAccount: name, account_type, institution, balance, currency, is_activeAccountTransaction: account_id, user_id, amount, description, category, transaction_dateService (
app/services/accounts.py):create_account()— with type validation and balance initializationget_account()— single account with recent transactionslist_accounts()— with optional active-only filterget_overview()— total balance, per-type breakdown, account countupdate_account()— partial update of allowed fieldsdeactivate_account()— soft-delete (is_active=False)add_transaction()— with auto balance update on the accountlist_transactions()— paginated per-account transaction listingRoutes (
app/routes/accounts.py):POST /accounts— create account with validationGET /accounts— list accounts (active_only param)GET /accounts/overview— multi-account dashboard (total, per-type, count)GET /accounts/<id>— single account with recent transactionsPATCH /accounts/<id>— update accountDELETE /accounts/<id>— deactivate accountPOST /accounts/<id>/transactions— add transaction (auto-updates balance)GET /accounts/<id>/transactions— list transactionsSchema (
app/db/schema.sql):financial_accountsandaccount_transactionstablesaccount_typeenum typeTests
11 tests covering:
Closes #132