-
Notifications
You must be signed in to change notification settings - Fork 3.7k
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 Twilio: fix expected records and docker image tag #26675
Source Twilio: fix expected records and docker image tag #26675
Conversation
Before Merging a Connector Pull RequestWow! What a great pull request you have here! 🎉 To merge this PR, ensure the following has been done/considered for each connector added or updated:
If the checklist is complete, but the CI check is failing,
|
/test connector=connectors/source-twilio
Build PassedTest summary info:
|
/test connector=connectors/source-twilio
Build FailedTest 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.
Please add link to failed action to PR description.
/test connector=connectors/source-twilio
Build PassedTest summary info:
|
…6675) * Source Twilio: fix acceptance tests * Source Twilio: fix acceptance tests * Source Twilio: fix dockerfile imagetag version * Update expected_records.jsonl
What
executions
steram became empty for some reason. After some manipulations a reocrd become present in response. There is also new fieldsubresource_uris
in incoming_phone_numbers stream. Docker image tag was not updated.How
Made new execution record, updated expected records and schema for incoming_phone_numbers stream. Updated docker image tag