Skip to content

Conversation

@bweston92
Copy link
Contributor

No description provided.

@amacneil
Copy link
Contributor

amacneil commented Apr 8, 2014

Looks good. Can we perhaps rename the methods to setApply3DSecure() to match the parameter name? Also we may as well add setApplyAVSCV2() at the same time.

@bweston92
Copy link
Contributor Author

Yeah that would be fine, want me to update it or you going todo it, I only added the 2 I required for my application.—
Sent from Mailbox for iPhone

On Tue, Apr 8, 2014 at 11:50 PM, Adrian Macneil [email protected]
wrote:

Looks good. Can we perhaps rename the methods to setApply3DSecure() to match the parameter name? Also we may as well add setApplyAVSCV2() at the same time.

Reply to this email directly or view it on GitHub:
#6 (comment)

@bweston92
Copy link
Contributor Author

Just done and committed. Let me know when you can pull as I need this to carry on :)

Update for method name on setApply3DAuth and added method setApplyAVSCV2
method.
@amacneil amacneil closed this in c902e94 Apr 10, 2014
@amacneil
Copy link
Contributor

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants