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

Source Freshdesk: refactor to use the CDK #7764

Closed
vitaliizazmic opened this issue Nov 8, 2021 · 0 comments · Fixed by #8682
Closed

Source Freshdesk: refactor to use the CDK #7764

vitaliizazmic opened this issue Nov 8, 2021 · 0 comments · Fixed by #8682

Comments

@vitaliizazmic
Copy link
Contributor

The Freshdesk connector currently uses an old pattern, we should migrate it to use the new CDK. It's part of #6164.

Describe the solution you’d like

see #6164 for clearer instructions

@vitaliizazmic vitaliizazmic added the type/enhancement New feature or request label Nov 8, 2021
@vitaliizazmic vitaliizazmic added this to the Connectors 2021-11-12 milestone Nov 8, 2021
@vitaliizazmic vitaliizazmic self-assigned this Nov 8, 2021
@sherifnada sherifnada added the area/connectors Connector related issues label Nov 15, 2021
vitaliizazmic added a commit that referenced this issue Dec 9, 2021
vitaliizazmic added a commit that referenced this issue Dec 14, 2021
* Source Freshdesk #7764 - migrate to the CDK

* Source Freshdesk #7764 - acceptance tests

* Source Freshdesk #7764 - update state

* Source Freshdesk #7764 - fixing according to PR

* Source Freshdesk #7764 - fixing client according to PR

* Source Freshdesk #7764 - bump version and update docs
schlattk pushed a commit to schlattk/airbyte that referenced this issue Jan 4, 2022
* Source Freshdesk airbytehq#7764 - migrate to the CDK

* Source Freshdesk airbytehq#7764 - acceptance tests

* Source Freshdesk airbytehq#7764 - update state

* Source Freshdesk airbytehq#7764 - fixing according to PR

* Source Freshdesk airbytehq#7764 - fixing client according to PR

* Source Freshdesk airbytehq#7764 - bump version and update docs
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging a pull request may close this issue.

4 participants