add service proxy chnages

J7mbo commented 7 years ago

Hi @eric-urban, I follow these merges closely. I have a few Q's if you don't mind?

I know this is only in a 0.0* version but is there a roadmap for a stable API - 1.0? Surely MS has a roadmap for your work, are you at liberty to divulge estimates (I'm not after deadlines, more a stable API that is safe to use - i.e. your code handles all the possible SOAP errors etc).
Do you accept PRs? I ask because I made a PR a while back and it was overwritten with your own. Is this an MS policy or just a passing one?

eric-urban commented 7 years ago

Hi @J7mbo -

With respect to stabilizing the SDK we are working on a longer term plan. Originally we only planned to release the proxy classes for samples that used to be hosted on MSDN code gallery. The previous samples release process was cumbersome because clients had to download and extract the ZIP manually, etc. As you are already familiar the process is streamlined via composer and packagist. Also of course Github collaboration is something we hope to leverage.

So yes, we would love to hear your feedback! Are there any specific feature requests that you have e.g. BulkServiceManager or ReportingServiceManager (similar to .NET SDK classes)? Let me know if you want to jump on a call and I can set something up, or if you prefer to open issues via Github that is great also.

Best regards, Eric

chiragvels commented 7 years ago

Hi @eric-urban ,

Yes, both the option are working now with the composer.

Thanks,

BingAds / BingAds-PHP-SDK

add service proxy chnages #22