Skip to content
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

Add missing $options reference in updateRecords method #49

Open
wants to merge 84 commits into
base: kaioken
Choose a base branch
from

Conversation

kaioken
Copy link
Member

@kaioken kaioken commented Oct 13, 2020

No description provided.

JimmyBeats and others added 30 commits June 3, 2016 05:34
Add missing $options reference in updateRecords method
Allow to add empty string values and fix some typos
Added const for Modules to avoid mistype, ex: setModule(ZohoClient::MODULE_LEADS);
…us constraints and allow semver installing of dev version
Add options param to convertLead
Doc update for supportEu pull request #28
support zoho.eu domain by adding setEuDomain method to ZohoClient
support getRecordById getting multiple records if passed array of ids
Update `composer.json` in order to simplify autoload, use non ambiguous constraints and allow semver installing of dev version
return XML container name based on called class
Fixing PSR-4 autoload folder location.
@todo
Copy link

todo bot commented Oct 13, 2020

Need to convert json data from Zoho API v2 to the same array given in our Zoho CRM SDK

* @todo Need to convert json data from Zoho API v2 to the same array given in our Zoho CRM SDK
*/
protected function parseResponse()
{
if ($this->method == 'get' && ($this->module == 'Contacts' || $this->module == 'Accounts' || $this->module == 'Potentials')) {
$this->parseResponseGetRecords();


This comment was generated by todo based on a todo comment in 4ef8d2b in #49. cc @mctekk.

@todo
Copy link

todo bot commented Oct 13, 2020

Give more options on default header by passing an array.

* @todo Give more options on default header by passing an array.
*/
public function getDefaultHeaders()
{
$this->generateAccessTokenByRefreshToken();
return [


This comment was generated by todo based on a todo comment in 4ef8d2b in #49. cc @mctekk.

@todo
Copy link

todo bot commented Oct 13, 2020

Modify createResponse so that it gives the same response regardless of how the response is structured

* @todo Modify createResponse so that it gives the same response regardless of how the response is structured
* @return Response
*/
protected function call($method, $params = [], $data = [], $options = [])
{
$defaultHeaders = $this->getDefaultHeaders();


This comment was generated by todo based on a todo comment in 4ef8d2b in #49. cc @mctekk.

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.