Curl options can now be set in the adapter class

[aleitner]

No description provided.

[JoshuaEstes]

doc block comments

[JoshuaEstes]

Looks ok to me, I would like to see some type of code coverage within the foreach loop to make sure it does what it's suppose to do. Other than that, I think this is ok to merge in.

[ionux]

Has this been regression tested with any of our existing plugins?

[aleitner]

I was able to create an invoice and pair using command line. I will move the card to build and do regression testing before it gets moved back to verify.

Alex Leitner
alex@bitpay.com

On Dec 1, 2014, at 2:28 PM, Rich Morgan notifications@github.com wrote:

Has this been regression tested with any of our existing plugins?

—
Reply to this email directly or view it on GitHub #126 (comment).

[aleitner]

Paul and I got stuck going down the Rabbit hole trying to figure this out today. We having been trying to create customizable options that also make it easy to change from Livenet/Testnet to using a local instance of bitpay. The way we figured it out requires a Network object to be created before an Adapter Object. So we may just create a config file that has constants for set already. And you would change these constants instead of using all these different classes and setting properties in each object