Skip to content

Add bank sync connector architecture#1179

Closed
sayuru-akash wants to merge 2 commits into
rohitdash08:mainfrom
sayuru-akash:feat/bank-sync-connectors
Closed

Add bank sync connector architecture#1179
sayuru-akash wants to merge 2 commits into
rohitdash08:mainfrom
sayuru-akash:feat/bank-sync-connectors

Conversation

@sayuru-akash
Copy link
Copy Markdown

Summary

  • add a pluggable bank connector interface and registry with a built-in mock connector
  • add bank connection import and refresh endpoints with sync-run tracking and external transaction dedupe
  • add schema/runtime compatibility tables, OpenAPI docs, README notes, and Redis fallback resilience for local/dev auth and cache flows

Bounty

Addresses #75 (Bank Sync Connector Architecture, bounty).

Tests

  • PYTHONPATH=. .venv311/bin/pytest tests
  • .venv311/bin/black --check app tests
  • .venv311/bin/flake8 app tests

@sayuru-akash sayuru-akash requested a review from rohitdash08 as a code owner June 4, 2026 15:07
@sayuru-akash sayuru-akash marked this pull request as draft June 4, 2026 15:12
@sayuru-akash sayuru-akash marked this pull request as ready for review June 4, 2026 15:12
@sayuru-akash
Copy link
Copy Markdown
Author

Update pushed to better match the Indian AA/API-provider requirement from the issue discussion.\n\nAdded a generic account_aggregator connector alongside mock so provider-specific adapters can store provider/account/consent references and import normalized partner-portal transaction payloads without putting live credentials in app config. Updated OpenAPI/README and added coverage for the normalized AA payload path.\n\nVerification after the update:\n- PYTHONPATH=. .venv311/bin/pytest tests -> 29 passed\n- .venv311/bin/black --check app tests -> passed\n- .venv311/bin/flake8 app tests -> passed\n- git diff --check -> passed

@sayuru-akash sayuru-akash closed this by deleting the head repository Jun 4, 2026
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.

1 participant