asciisd / zoho-v3

Laravel Zoho API V3 Package
MIT License
16 stars 17 forks source link

Please Add #10

Closed 1337erik closed 1 year ago

1337erik commented 1 year ago

I am manually adding a sort_by and sort_direction to the package, its super simple.

Would highly recommend adding this but I dont know how to do a pull request

image
aemaddin commented 1 year ago

Hi @1337erik

Thanks for the contribution. You can make a pull request by following these steps:

1- take a fork from the project on your account 2- clone your forked project on your local machine 3- make your edits 4- push your edits to the GitHub 5- on your forked version, you can make a pull request to the original project

1337erik commented 1 year ago

Hey man hows it going - I have an issue with the ZOHO api integration - I posted a few times to the Issues board https://github.com/asciisd/zoho-v3/issues/11

So I have a local appo and a production app now, Zoho only allows creating 1 Self Client and if both try to grant using the Self Client it will invalidate the other one. That being said it seems like I need to use a Server Application API form the API console.

It was working for me for like 30-45 minutes, and then it just kept giving me "invalid token" errors. Is the refresh token process not working for the v3 package? Do you have any insight into how this can be added or fixed?

On Fri, Dec 30, 2022 at 3:59 AM Amr Ahmed @.***> wrote:

Hi @1337erik https://github.com/1337erik

Thanks for the contribution. You can make a pull request by following these steps:

1- take a fork from the project on your account 2- clone the project on your local machine 3- make your edits 4- push your edits to the GitHub 5- on your forked version you can make a pull request to the original project

— Reply to this email directly, view it on GitHub https://github.com/asciisd/zoho-v3/issues/10#issuecomment-1367801756, or unsubscribe https://github.com/notifications/unsubscribe-auth/AE3HSFJBUPQ5TRFUPNRAK6DWP2P6RANCNFSM6AAAAAATMRNLSQ . You are receiving this because you were mentioned.Message ID: @.***>

upsmod commented 1 year ago

@1337erik that is another issue #14 please follow that