Skip to content

Conversation

@hscabral
Copy link

  • Fixed API endpoints for creating (POST) Sections and Categories - endpoint had /translation.json and according to Zendesk documentation, it should be /translations.json

  • Added description attribute to Section class

Copy link
Contributor

@aheritier aheritier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

The end-point issue looks to come from PR#180
Not sure if something changed on zendesk side
@assafko do you remember about this?

aheritier
aheritier previously approved these changes Oct 17, 2018
Copy link
Contributor

@aheritier aheritier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Proposed changes are LGTM

@aheritier
Copy link
Contributor

I don't think the test failure was related to the issue

Copy link
Contributor

@aheritier aheritier left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@aheritier aheritier merged commit c3f9a0c into cloudbees-oss:master Jun 10, 2019
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.

2 participants