search

c12i / mpesa-rust

A MPESA API sdk in Rust
https://c12i.github.io/mpesa-rust/mpesa/index.html
MIT License
43 stars 14 forks source link

Update bill manager to latest api docs and use derive builder. #92

Open crispinkoech opened 9 months ago

crispinkoech commented 9 months ago

Closes https://github.com/c12i/mpesa-rust/issues/87 and https://github.com/c12i/mpesa-rust/issues/91; still work in progress.

c12i commented 9 months ago

Hey @crispinkoech should I review while still in draft?

crispinkoech commented 9 months ago

Naah, not yet. Will mark it as ready (and/or ping you) once it's good for review.

On Wed, 29 Nov 2023 at 17:39, Collins Muriuki @.***> wrote:

Hey @crispinkoech https://github.com/crispinkoech should I review while still in draft?

— Reply to this email directly, view it on GitHub https://github.com/c12i/mpesa-rust/pull/92#issuecomment-1832022808, or unsubscribe https://github.com/notifications/unsubscribe-auth/AGIV6HTTX26DE2RLDCIK6PLYG5CLLAVCNFSM6AAAAAA74NPIKSVHI2DSMVQWIX3LMV43OSLTON2WKQ3PNVWWK3TUHMYTQMZSGAZDEOBQHA . You are receiving this because you were mentioned.Message ID: @.***>

crispinkoech commented 1 month ago

Oooft, this MR is too out of sync with the current master commit; @itsyaasir @c12i I suggest I close this and start some new work off of master?

Also, when was the cancel_bulk_invoice.rs module/functionality removed? Can't seem to find a merge request that deleted it.