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

Requests: API to implement #8

Open
schmorrison opened this issue Dec 17, 2018 · 11 comments
Open

Requests: API to implement #8

schmorrison opened this issue Dec 17, 2018 · 11 comments
Labels
enhancement New feature or request

Comments

@schmorrison
Copy link
Owner

Leave a comment in this issue if you would like to see an API, or specific function of an API implemented.

@schmorrison schmorrison added the enhancement New feature or request label Dec 17, 2018
@bondar-pavel
Copy link
Contributor

Hello, thanks for your work.
We already using Zoho CRM in our projects.

And also would like to see Zoho Subscriptions API.
More specifically currently we would like to manage Subscriptions&Invoices:
https://www.zoho.com/subscriptions/api/v1/#Subscriptions
https://www.zoho.com/subscriptions/api/v1/#Invoices

@schmorrison
Copy link
Owner Author

@bondar-pavel noted. I will see what I can do about that. Looking over the API docs there shouldn't be any surprises. Feel free to have a run at it, submit a PR, and I can make changes where necessary. Otherwise, it has been added to the roadmap.

BTW, thanks for the endorsement. :)

@bondar-pavel
Copy link
Contributor

Added draft pull request with List and Get Subscription for now: #25

@ysahil97
Copy link
Contributor

Hello, I appreciate your work in building this comprehensive Zoho go API library

I have been working in a startup which uses golang. We wanted to have Zoho bookings (link) implementation to be used for our purpose. Currently I have forked your repo and written the Zoho booking API implementations. Could I submit a PR for the same?

@schmorrison
Copy link
Owner Author

Absolutely @ysahil97 !!

It always take a few days for me to review be warned. :)

@ashishsnigam
Copy link
Contributor

@schmorrison I can again contribute to this repo. Please feel free to assign me any task. I will do it in my free time.

@schmorrison
Copy link
Owner Author

schmorrison commented Aug 8, 2021

I'll get some kind of a roadmap together. Lots of little features I'd like to add but just don't have enough time. I'd love the help.

I think implementing endpoints can be a little boring, so the big goal is to make contribution as simple as possible.

To that end, here are a couple ideas:

I started work many months ago on a code generation branch. Not sure if it's a totally appropriate approach though.

More recently I started working on a branch to add unit tests, something I am very unfamiliar with. I figure it'd be great if users could contribute a unit test along with their endpoint contribution. Setting up a way to test the endpoint would be amazing. A mock HTTP server/client and inspect the requests/responses, (requests to Zoho shouldn't be performed during tests I figure).

Finally, Zoho has custom fields. I'd like to figure out how we might be able to use struct embedding to encode/decode custom fields along with the standard fields.

Start an issue if you want to take anything on, we can discuss the details there.

Again, thanks for the offer!!

@ashishsnigam
Copy link
Contributor

Hi @schmorrison Zoho officially does not support Golang based SDKs. May be we can start working towards that direction taking inspiration from Python SDK and make this repo as unofficial Golang SDK for Zoho.

Regarding unit tests, for new apis unit tests should be written along with API wrapper. And for already developed apis it's a work to be completed. I propose, you should first create TODO tasks in this repo itself. Refactoring will always be there as and when needed.

@schmorrison
Copy link
Owner Author

@ashishsnigam Ok, great.

I think if we build something good, that regard will follow. I'll work on a todo list and publish it to the repo, can I send you an email directly?

@ashishsnigam
Copy link
Contributor

@schmorrison sure.

@schmorrison
Copy link
Owner Author

@ashishsnigam checkout issue #33

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

4 participants