-
Notifications
You must be signed in to change notification settings - Fork 9
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
Basic test cases carrier billing payment #174
Conversation
🦙 MegaLinter status: ✅ SUCCESS
See detailed report in MegaLinter reports |
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.
Thanks Pedro for the effort.
No major issue from my perspective but some small things that I have caught.
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.
Thanks Pedro for the corrections.
I went thru and this is OK for me.
Thanks for the review Ludovic! |
What type of PR is this?
What this PR does / why we need it:
Basic Tests for Carrier Billing API
Which issue(s) this PR fixes:
Partially Fixes #159
Special notes for reviewers:
Tests for Carrier Billing Refund will be uploaded in a separate PR
Changelog input
Additional documentation
N/A