-
Notifications
You must be signed in to change notification settings - Fork 4.3k
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: fixes to make acceptance tests pass #12183
Conversation
/test connector=connectors/source-twilio
|
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 @alafanechere can you create an issue to add data to the empty streams in the future?
Done: #12232 |
/publish connector=connectors/source-twilio
|
What
Acceptance tests run on
source-twilio
were failing. I try to make the required fix in this PR.Closes #9225
How
streams
property in thecalls
stream'ssubresource_uris
(probably a new subresource that is now returned by the Twilio API.conference_participants
streammessage_medias
andconferences
as empty stream in the acceptance test config as our sandbox account does not have data for these streams