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

Feature/fix/expiry time access token #58

Open
wants to merge 18 commits into
base: master
Choose a base branch
from
Open

Feature/fix/expiry time access token #58

wants to merge 18 commits into from

Conversation

jratzenboeck
Copy link

Changes

  • Convert expiry duration of access token to milliseconds before adding to final expiry time.

Jürgen Ratzenböck and others added 18 commits October 18, 2018 09:37
…dex 1

This allows to have an editorconfig with insert_final_new_line and prevents
a nasty bug to track down.
…ing them

PHP throws an exception when you call non-static methods from within a static
method. This was done until this change.
The sandbox flag comes into the zoho api as boolean flag from our config
file but was interpreted by the official zcrm php sdk as string ("true" or "false")
which makes no sense. Thus, interpet it as boolean value to decide whether to use
sandbox or production.
Remove array filter to allow syncing of null values
Bugfix Check if is array before calling the count function
JOBS-526 Replaces deprecated offset access syntax
…zoho-api

Bugfix/jobs 845/zoho crm not call zoho api
The expiry time is returned in seconds from the refresh token call but
the current timestamp is retrieved in milliseconds. Thus we have to convert
the duration first to milliseconds before adding it to the final expiry time.
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.

3 participants