Skip to content

feat(b2p): add sync Business to Pochi client - #176

Open
LewisJassy wants to merge 2 commits into
Byte-Barn:masterfrom
LewisJassy:feature/Business_to_Pochi
Open

feat(b2p): add sync Business to Pochi client#176
LewisJassy wants to merge 2 commits into
Byte-Barn:masterfrom
LewisJassy:feature/Business_to_Pochi

Conversation

@LewisJassy

Copy link
Copy Markdown
Contributor

Adds a synchronous Business to Pochi (B2Pochi) client for paying a customer's Pochi La Biashara wallet via /mpesa/b2pochi/v1/paymentrequest.

  • New feature (non-breaking change that adds functionality)

  • Tests (addition of tests)

  • Unit tests in tests/unit/b2p/test_b2p.py (26 passed)

  • Ruff lint/format on mpesakit/b2p and tests/unit/b2p

  • mypy mpesakit/b2p

Coverage includes request validation (CommandID, PartyB normalization, Remarks/Occassion length), the acknowledgement response, nested Daraja success/failure callbacks, and timeout callback schemas.
#159

Daraja B2Pochi payments need a typed request/callback layer so
disbursements to Pochi La Biashara can be initiated and reconciled
without hand-parsing payloads.
Added support for B2Pochi payments, including a new callback processing method and service integration. Updated README to include usage examples and details for the B2Pochi service.

@RafaelJohn9 RafaelJohn9 left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hello @LewisJassy , Thank you for an awesome PR, we are off to a great start, few changes to consider though

Otherwise this is a great start.

Note

Apologies for the late review, if you are done with changes and ready for rereview you can always request it or tag me 🤝 .

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.

2 participants