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.

Sign up for GitHub

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

Jump to bottom

Source Google Directory: refactor to use the CDK #7415

Closed
vitaliizazmic opened this issue Oct 27, 2021 · 0 comments · Fixed by #7464
Closed

Source Google Directory: refactor to use the CDK #7415

vitaliizazmic opened this issue Oct 27, 2021 · 0 comments · Fixed by #7464
Assignees
@vitaliizazmic
Labels
area/connectors Connector related issues connectors/source/google-directory connectors/sources-api type/enhancement New feature or request
Milestone
Connectors 2021-1...

Comments

@vitaliizazmic
Copy link
Contributor

The Google Directory 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 type/enhancement New feature or request area/connectors Connector related issues labels Oct 27, 2021
@vitaliizazmic vitaliizazmic self-assigned this Oct 27, 2021
@vitaliizazmic vitaliizazmic changed the title Source Google Directory: Refactor to use the CDK Source Google Directory: refactor to use the CDK Oct 27, 2021
@vitaliizazmic vitaliizazmic mentioned this issue Oct 27, 2021
Closed
36 tasks
@VasylLazebnyk VasylLazebnyk added this to the Connectors 2021-10-29 milestone Oct 28, 2021
vitaliizazmic added a commit that referenced this issue Oct 28, 2021
@vitaliizazmic
Source Google Directory #7415 - migrate to the CDK
fc65f2e
@vitaliizazmic vitaliizazmic mentioned this issue Oct 28, 2021
Merged
10 tasks
vitaliizazmic added a commit that referenced this issue Nov 2, 2021
@vitaliizazmic
Source Google Directory #7415 - fix according to PR review
be7d0d4
vitaliizazmic added a commit that referenced this issue Nov 2, 2021
@vitaliizazmic
Source Google Directory #7415 - added etag and lastLoginTime to ignor…
40d6fef 
…ed fields for full refresh acceptance test
vitaliizazmic added a commit that referenced this issue Nov 2, 2021
@vitaliizazmic
Source Google Directory #7415 - fix full refresh acceptance test config
9d32b41
vitaliizazmic added a commit that referenced this issue Nov 2, 2021
@vitaliizazmic
Source Google Directory #7415 - bump version
b61c07f
vitaliizazmic added a commit that referenced this issue Nov 2, 2021
@vitaliizazmic
🎉 Source Google Directory: migrate to the CDK
d37e00e 
* Source Google Directory #7415 - migrate to the CDK

* Source Google Directory #5190 - fix timeout error

* Source Google Directory #7415 - fix according to PR review

* Source Google Directory #7415 - added etag and lastLoginTime to ignored fields for full refresh acceptance test

* Source Google Directory #7415 - fix full refresh acceptance test config

* Source Google Directory #7415 - bump version
lmossman pushed a commit that referenced this issue Nov 3, 2021
@vitaliizazmic @lmossman
🎉 Source Google Directory: migrate to the CDK
03f4e49 
* Source Google Directory #7415 - migrate to the CDK

* Source Google Directory #5190 - fix timeout error

* Source Google Directory #7415 - fix according to PR review

* Source Google Directory #7415 - added etag and lastLoginTime to ignored fields for full refresh acceptance test

* Source Google Directory #7415 - fix full refresh acceptance test config

* Source Google Directory #7415 - bump version
This was referenced Nov 4, 2021
Merged
Merged
schlattk pushed a commit to schlattk/airbyte that referenced this issue Jan 4, 2022
@vitaliizazmic @schlattk
🎉 Source Google Directory: migrate to the CDK
fca2964 
* Source Google Directory airbytehq#7415 - migrate to the CDK

* Source Google Directory airbytehq#5190 - fix timeout error

* Source Google Directory airbytehq#7415 - fix according to PR review

* Source Google Directory airbytehq#7415 - added etag and lastLoginTime to ignored fields for full refresh acceptance test

* Source Google Directory airbytehq#7415 - fix full refresh acceptance test config

* Source Google Directory airbytehq#7415 - bump version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@vitaliizazmic vitaliizazmic

Labels
area/connectors Connector related issues connectors/source/google-directory connectors/sources-api type/enhancement New feature or request
Projects
None yet
Milestone
Connectors 2021-11-12
Development

Successfully merging a pull request may close this issue.

🎉 Source Google Directory: migrate to the CDK airbytehq/airbyte
4 participants
@sherifnada @VasylLazebnyk @vitaliizazmic @igrankova