-
Notifications
You must be signed in to change notification settings - Fork 26
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
Has been added the Profile service (CRUD) #2
Conversation
@@ -4,7 +4,7 @@ class AuthorizationResponse | |||
include Utils | |||
|
|||
FIELDS = [:respstat, :retref, :account, :token, :amount, :merchid, :respcode, | |||
:resptext, :respproc, :avsresp, :cvvresp, :authcode, :commcard, :profileid] | |||
:resptext, :respproc, :avsresp, :cvvresp, :authcode, :commcard, :profileid, :acctid] |
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.
Does this field come over when getting an Auth response from CardConnect's API? It does not appear to be in their documentation unless I am missing something.
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.
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.
Noted. LGTM
Thanks for the PR! Would you be willing to add some tests and update the documentation for Profile service in the README? |
about tests and documentation... I'll do it |
Thanks! |
@AndreiBujenitsa Hey, I found some time to do the tests for this so I will merge it and handle that part. Thanks for the contribution! |
No description provided.