Skip to content

fix: Bank Sync Connector Architecture#948

Open
bounty-bot-beep wants to merge 1 commit intorohitdash08:mainfrom
bounty-bot-beep:bounty/issue-75
Open

fix: Bank Sync Connector Architecture#948
bounty-bot-beep wants to merge 1 commit intorohitdash08:mainfrom
bounty-bot-beep:bounty/issue-75

Conversation

@bounty-bot-beep
Copy link
Copy Markdown

Summary

Resolves #75: Bank Sync Connector Architecture

All 36 tests pass (22 existing + 14 new).

Summary of changes for issue #75 (Bank Sync Connector Architecture):

  • app/services/bank_sync/ — new pluggable package

    • base.py: BankConnector abstract interface with connect and fetch_transactions, plus BankAccountInfo / BankTransaction dataclasses and ConnectorError
    • registry.py: register_connector decorator, get_connector, list_connectors
    • mock.py: MockConnector self-registers as "mock" with deterministic accounts and transactions
    • __init__.py: re-exports the public surface and eagerly imports the mock
  • app/models.py — added BankAccount model (provider, external_id, name, account_type, currency, last_synced_at)

  • app/db/schema.sql — added bank_accounts table

  • **`app/routes/ba


Closes #75

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Bank Sync Connector Architecture

1 participant