-
Notifications
You must be signed in to change notification settings - Fork 3.9k
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-clazar: removed redundant columns from streams, added documentation for analytics #41657
✨ source-clazar: removed redundant columns from streams, added documentation for analytics #41657
Conversation
…shant-mittal9/source-clazar-exclusions
The latest updates on your projects. Learn more about Vercel for Git ↗︎
|
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@prashant-mittal9 let us know when this is ready to review.
…shant-mittal9/source-clazar-schema-upgrade
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It is missing the update the connector version in the pyproject and metadata file.
@natikgadzhi we need to /approve this. Can you double check it? The file is a valid connector in Builder. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
Hello your feedback matters a lot to us. Can you spare just 2 minutes to complete a survey? We’re dedicated to making the contributor experience even better, and your input is key to achieving excellence. Thank you for helping us improve! |
What
How
Review guide
User Impact
Can this PR be safely reverted and rolled back?