Skip to content

feat: Bank Sync Connector Architecture (Issue #75)#304

Open
Abu1982 wants to merge 1 commit intorohitdash08:mainfrom
Abu1982:feature/bank-sync-connector
Open

feat: Bank Sync Connector Architecture (Issue #75)#304
Abu1982 wants to merge 1 commit intorohitdash08:mainfrom
Abu1982:feature/bank-sync-connector

Conversation

@Abu1982
Copy link

@Abu1982 Abu1982 commented Mar 4, 2026

Summary

Implements a pluggable bank connector architecture as requested in Issue #75.

Changes

  1. Bank Connector Interface (�pp/services/bank_connectors/init.py):

    • Abstract BankConnector base class with interface methods
    • BankAccount and Transaction dataclasses
    • Factory function for connector registration
  2. Mock Connector (�pp/services/bank_connectors/mock.py):

    • Full mock implementation for testing/development
    • Returns sample accounts and transactions
  3. API Routes (�pp/routes/bank_connector.py):

    • GET /bank/accounts - List connected accounts
    • GET /bank/accounts//transactions - Get transactions
    • POST /bank/accounts//refresh - Refresh account balance
    • GET /bank/connectors - List available connector types
    • POST /bank/test-connection - Test connection

Acceptance Criteria Met

  • ✅ Connector interface
  • ✅ Import & refresh support
  • ✅ Mock connector included

Testing

The mock connector provides:

  • 3 mock accounts (checking, savings, credit)
  • Sample transactions for the last 30 days

To test locally:
`�ash

With mock connector (no API key needed)

curl http://localhost:5000/bank/accounts?connector_type=mock
`

Closes #75

- Add pluggable bank connector interface with base class
- Add mock connector for testing and development
- Add API routes for account management and transaction import
- Include import and refresh functionality as per issue rohitdash08#75
@Abu1982 Abu1982 requested a review from rohitdash08 as a code owner March 4, 2026 20:40
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