Skip to content

Introduce objects/methods for dealing with voice call recordings #40

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

Conversation

mathieuk
Copy link

@mathieuk mathieuk commented Dec 4, 2019

[CHANGES] Namespace for Call object to MessageBird::Voice
[ADDS] New MessageBird::Voice::Base class for voice-related entities
[CHANGES] Request for call_list goes through MessageBird::VoiceClient
[CHANGES] CallList -> MessageBird::Voice::List
[ADDS] call_leg_list method + test
[ADDS] call_leg_recordings_list method + test
[ADDS] call_leg_recording_view method + test
[ADDS] call_leg_recording_download method + test
[ADDS] MessageBird::HttpClient::request_block method to perform a request and handle downloading in the provided Ruby block
[ADDS] New examples for calls, call_leg and call_leg_recordings

This will clash with #38 so I'm waiting for that work to complete to finish this.

Mathieu Kooiman and others added 10 commits December 4, 2019 14:40
- [FIXES] the `call_list` method to access the proper endpoint
- [ADDS] the `call_leg_list` method to request the call legs
- [ADDS] the `call_leg_recording_list` to request the recordings for a leg
- [ADDS] the `call_leg_recording_view` to view the details for one recording
- [ADDS] the `call_leg_recording_download` call to download a recording
- [ADDS] tests for the new methods
- restore the libdir thing
- load messagebird/version in the main library
- Extract the `underscoring` func into `map_hash_elements_to_self`
- Make MessageBird::Voice::List use `map_hash_elements_to_self` for pagination block
@marcelcorso marcelcorso merged commit 2efa3ac into messagebird:master Dec 5, 2019
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.

3 participants