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

feat: Add GtheSheep variant tap-contentful #1446

Merged
merged 5 commits into from
Jul 24, 2023
Merged

Conversation

GtheSheep
Copy link
Contributor

No description provided.

@netlify
Copy link

netlify bot commented Jul 23, 2023

👷 Deploy request for meltano-hub pending review.

Visit the deploys page to approve it

Name Link
🔨 Latest commit ddb3147

@pnadolny13
Copy link
Contributor

@GtheSheep this is fantastic! What do you think about making this the default variant? I'm sure you looked at the existing variant so you'd probably know better but it looks like its very outdated and the settings arent even fully listed. I'd vote to make yours the default.

@GtheSheep
Copy link
Contributor Author

Hey @pnadolny13 - happy to! tbh I never got past installing the other one (it failed), and didn't look much into whether that was just a outdated package thing or a compatibility with Meltano thing, so reached straight for the SDK 😅

@pnadolny13 pnadolny13 enabled auto-merge (squash) July 24, 2023 16:54
@pnadolny13 pnadolny13 merged commit 65ce1ae into meltano:main Jul 24, 2023
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