forked from pluspeople/pesaPi
-
Notifications
You must be signed in to change notification settings - Fork 0
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pi #1
Merged
Merged
pi #1
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
A small typo preventing the parsing of paybill receipt numbers caused them to not be registered properly. This has been fixed.
MpesaPaybill transactions had an issue where they would trigger a push notification several time for the same payment. This left the receiving application to keep track of repeated transactions and was generally not so cool. The behaviour has been changed it will not only trigger a push callback upon creation of a new transaction. This may have to be examined further to avoid situations where existing transactions gets cancled/rolled-back by Safcom as they might have the same receipt id. (hence a double trigger is actually needed in that case).
When a statement from the Mpesa paybill platform return no results it will instead return a semi result stating "No records to display". This would result in an "unknown" transaction being registered. This flaw has been fixed - the html scraber now proberly detects and ignores these no result transaction rows. In addition initial work on Airtel Paybill, and mobile money in Somalia has been included.
I have been working on the Airtel Paybill support. The main http requests flow is now sorted - Airtel Paybill is now almost working.
The Airtel paybill code just managed to scrape it's first transactions and record them into PesaPi. There is still a bit of details to be sorted before it can be considered "fully operational" but all the pieces needed are now in place.
Support for Golis's Sahal mobile money system in Somalia has been added. Take note that as usual all amounts are listed in the lowest currency denomications As Sahal uses fractions of cents (aka uses 3 decimals) you have to divide the amounts by 1000 to get a "normal" representation. Sahal support has been marked as experimental, until we have a confirmed real life test by someone "on the ground". Thanks goes out to Saiid Cali for providing information about the system.
I have added support for MTN mobile money in Ghana. Only receiving money is supported at this particular time, and in general the support is experimental (untill properly tested). It's based on examples and information provided by Baba Musah
I have added initial support for Mpesa in DR Congo, at this point the support is considered experimental.
The recent "Move MPESA back to Kenya" aka Mpesa Generation 2 (G2) introduced some changes to the mpesa texts. This is the first fix to make pesapi understand the "updated" sms's - this needs to be tested in more detail but it was important to get any fixes out asap.
I have added initial support for MTN private accounts in Uganda. Currently I only know about the format of receiving money in a direct transfer, so the system is limited to this type of transaction. If anyone got more examples of MTN mobile money SMS's in Uganda then feel free to post them on the pesapi mailinglist.
I have made the data parsing more resiliant so that it does not fail as hard when the dateformat of a system changes. Obviously this is a two-edged tool - it will make installations more robust, but also make it easier for changes in dateformats to go by unnoticed._
Fixed the missing { at the beginning of the Parser for MTN uganda.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
No description provided.