search

intuit / QuickBooks-V3-PHP-SDK

Official PHP SDK for QuickBooks REST API v3.0: https://developer.intuit.com/
Apache License 2.0
246 stars 246 forks source link

Added order_by to list of report options. #331

Closed chas688 closed 4 years ago

chas688 commented 4 years ago

I found out there is a bug in the v3 API for report service which throws an API error for sort_by modifier. However, using 'order_by' works in this case. sort_order works fine along with order_by, so I did not need to add anything else. I left sort_by and just added the order_by modifier to the reportService. Hope this helps someone else as well.

bsivalingam commented 4 years ago

Thanks @chas688! I would like to see if the API team can fix it and we don't have to work around. I will merge it or not based on what I hear from them.

chas688 commented 4 years ago

Yeah, It is clearly something wrong on their end, maybe you can make some headway with it in your position. Let me know. Thanks!

By the way, also submitted this to the support team if that helps as well: https://help.developer.intuit.com/s/question/0D54R000078pIa5SAE/in-qbo-api-when-using-reports-sortby-as-listed-in-the-api-docs-returns-an-error-however-orderby-seems-to-work

bsivalingam commented 4 years ago

Absolutely @chas688! I see Leela is following up on the forum, I will let her take this up with the API team, I'll made sure we solve for it. Will close the PR.