-
Notifications
You must be signed in to change notification settings - Fork 15
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
zendesk.tickets (major update), zendesk.chats (removed) #181
base: dev
Are you sure you want to change the base?
Conversation
d520e6a
to
8a3e42e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
We don't use package-lock.json
files yet. Please remove it.
53b22a0
to
181353b
Compare
181353b
to
ac1cb4e
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@DavidDurman quotas are missing, there should be at least some quotas set
The rate limits of Zendesk are very complicated and also depend on the account tier. I'm not sure how this can be implemented using our current quotas mechanism. See https://developer.zendesk.com/api-reference/introduction/rate-limits/#zendesk-suite-plan-limits |
|
… fields including attachments
…Ticket, FindTicketcoments, UpdateTicket, CreateUser, UpdateUser
…hats that was obsolete and unfinished
ac1cb4e
to
f5b4b0f
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
apply quotas in compoent.json files
Q: It's been under heavy discussions whether to move this connector under the
zendesk/
directory or leave it as is inzendesktickets/
.This PR also updates the
zendesk.chats
connector and fixes https://github.com/clientIO/appmixer-components/issues/1873