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: Braintree connector integration #30
Conversation
26c2d86
to
1bb3a8c
Compare
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.
I've requested some minor changes.
I'm assuming this is a WIP PR. Let me know if it is indeed WIP, I'll update the title to indicate the same.
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.
Please make the suggested changes
Signed-off-by: Sahebjot Singh <sahebjot94@gmail.com>
remove unnecessary comments fix imports part of issue #56
5970745
to
c83d916
Compare
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.
Minor set of changes, please address them. Other than that, looks good to me!
Also, could you please update the checklist items in #56 to indicate the PRs in which the checked items were included?
Something along the lines of the following snippet should be good:
### Payment Methods
- [x] Card #30
… minor formatting part of #56
02b0a84
to
5f00ba4
Compare
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.
Looks good to me!
I'm assuming the rest of the items will be taken up in separate PRs.
yes separate PRs |
Type of Change
Description
Added the Braintree connector.
Additional Changes
added Braintree base url in config files
Motivation and Context
To add Braintree connector
How did you test it?
Did a successful payment on Braintree sandbox.
Checklist
cargo +nightly fmt
cargo clippy