-
Notifications
You must be signed in to change notification settings - Fork 4.1k
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 Stripe: Add missing field (cancel_at) to the Subscriptions stream #22898
🐛 Source Stripe: Add missing field (cancel_at) to the Subscriptions stream #22898
Conversation
/test connector=connectors/source-stripe
Build PassedTest summary info:
|
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.
All looks good - just remove the version bumps on those two files!
airbyte-config/init/src/main/resources/seed/source_definitions.yaml
Outdated
Show resolved
Hide resolved
Running the build on a non-forked branch here: #23452 |
Closing this in favor of the non-forked PR to make sure we see that the build is passing. Waiting on a review from @airbytehq/connector-operations: |
What
The change solves this issue where the
cancel_at
field is not showing on the UI nor the final table.How
I added the field to the configured catalog and schema.
Recommended reading order
🚨 User Impact 🚨
No breaking changes
Pre-merge Checklist
Updating a connector
Community member or Airbyter
airbyte_secret
./gradlew :airbyte-integrations:connectors:<name>:integrationTest
.README.md
bootstrap.md
. See description and examplesdocs/integrations/<source or destination>/<name>.md
including changelog. See changelog exampleAirbyter
If this is a community PR, the Airbyte engineer reviewing this PR is responsible for the below items.
/test connector=connectors/<name>
command is passing/publish
command described here