-
Notifications
You must be signed in to change notification settings - Fork 48
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
feat: remittances #72
base: master
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good job so far, just take a look at the comments I have left
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
remember to squash the commits and use conventional commit messages (prefixed with feat:
or fix:
) to make a release and push to npm happen
yes i did. `referenceId` is not optional.
…---
Luigi Morel.
Software Engineer
Tel: +256 704 916 696 (WhatsApp only)
On Wed, Apr 20, 2022, 14:31 Ernest Okot ***@***.***> wrote:
***@***.**** commented on this pull request.
------------------------------
In src/errors.ts
<#72 (comment)>:
> @@ -171,7 +172,7 @@ export function getError(code?: FailureReason, message?: string) {
return new UnspecifiedError(message);
}
-export function getTransactionError(transaction: Payment | Transfer) {
+export function getTransactionError(transaction: Payment | Transfer | Remit) {
const error: MtnMoMoError = getError(transaction.reason as FailureReason);
Did you check in the docs?
—
Reply to this email directly, view it on GitHub
<#72 (comment)>,
or unsubscribe
<https://github.com/notifications/unsubscribe-auth/ALR3BBP3A5T72BC7VJS35SDVF7TITANCNFSM5TPJSPOA>
.
You are receiving this because you authored the thread.Message ID:
***@***.***>
|
Add the remittance operation to SDK.
Fixes #14.
Please test and let me there's anything I'm missing.