Skip to content
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

PayerMessage and PayeeNote are required. #69

Open
mtvbrianking opened this issue Feb 8, 2022 · 3 comments
Open

PayerMessage and PayeeNote are required. #69

mtvbrianking opened this issue Feb 8, 2022 · 3 comments
Labels

Comments

@mtvbrianking
Copy link
Owner

mtvbrianking commented Feb 8, 2022

From a user in production.

If either PayerMessage or PayeeNote is empty, you will get an Internal Server Error.

TODO:

Improve Collection::requestToPay to require these two

@warmwhisky
Copy link

Hi. I am trying to figure out what is appropriate to put in payerMessage and payerNote. In the API docs its says that it will be in the transaction history. But I also see something in PreApproval for payerMessage - The mesage that is shown to the approver. So this is throwing me a bit.

When I use your requestToPay function will the requestToPay show a message to the user on their device using payerMessage or will is go into transaction history?

Thanks for the great package by the way!

@mtvbrianking mtvbrianking changed the title PayerMessage and PayerNote are required. PayerMessage and PayeeNote are required. Mar 19, 2023
@mtvbrianking
Copy link
Owner Author

@warmwhisky the PayerMessage and PayeeNote will only be part of the transaction history.

MTN will render a different message to the user prompting them to enter their PIN to approve a debit of AMOUNT from initiator (your company) at a FEE and TAX

@warmwhisky
Copy link

@mtvbrianking Thank you for clearing that up for me.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants