Skip to content

Conversation

@tristan-vu
Copy link
Contributor

Description

This pull request adds support for creating ZIS (Zendesk Integration Services) Basic Auth connections through the API. The main changes include introducing a new interface to model the Basic Auth connection response and implementing a new method in the ZendeskApiService to create such connections.

ZIS Basic Auth Connection Support:

  • Added a new interface ICreateZisBasicAuthConnection in zendesk-integration-services.ts to represent the structure of a Basic Auth connection response.
  • Imported the new interface into zendesk-api-service.ts for use in the service layer.
  • Implemented a new method createZisBasicAuthConnection in ZendeskApiService to allow creating Basic Auth connections for a specified integration, including parameters for username, password, connection name, and allowed domain.

How to manually test

Include label

We are following this pattern to keep our package version updated. Please add the appropriate label to your Pull requests between the following:

  • Version: Minor

Acceptation criteria

  • Added the corrected label to my pull request
  • Added/updated tests impacted by the change
  • Documentation is up-to-date (README.md / INSTALL.md)
  • Manually tested?

github-actions[bot]
github-actions bot previously approved these changes Oct 14, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

LGTM! 🤖 🚀 🔥

Vico1993
Vico1993 previously approved these changes Oct 14, 2025
Copy link
Contributor

@Vico1993 Vico1993 left a comment

Choose a reason for hiding this comment

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

Looks good! but maybe we could rename createZisConnection ? To avoid confusion between Bearer token and Basic auth?

@github-actions github-actions bot dismissed their stale review October 15, 2025 17:36

Dismiss bot.

github-actions[bot]
github-actions bot previously approved these changes Oct 15, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

LGTM! 🤖 🚀 🔥

@tristan-vu tristan-vu force-pushed the create-basic-auth-connection branch from d07e0bd to 2ffeee6 Compare October 15, 2025 18:15
@github-actions github-actions bot dismissed their stale review October 15, 2025 18:15

Dismiss bot.

github-actions[bot]
github-actions bot previously approved these changes Oct 15, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

LGTM! 🤖 🚀 🔥

@github-actions github-actions bot dismissed stale reviews from Vico1993, cuneytcelebican, and themself October 16, 2025 12:42

Dismiss bot.

github-actions[bot]
github-actions bot previously approved these changes Oct 16, 2025
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

LGTM! 🤖 🚀 🔥

@tristan-vu tristan-vu force-pushed the create-basic-auth-connection branch from 0efdb4d to 9a8bc0a Compare October 16, 2025 17:37
@github-actions github-actions bot dismissed their stale review October 16, 2025 17:37

Dismiss bot.

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

LGTM! 🤖 🚀 🔥

@tristan-vu tristan-vu added this pull request to the merge queue Oct 16, 2025
Merged via the queue into main with commit 1f2bd6d Oct 16, 2025
6 of 7 checks passed
@tristan-vu tristan-vu deleted the create-basic-auth-connection branch October 16, 2025 18:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants