[ACTION] Zoom app, Zoom Phone's Get Call Logs Action

[pipedream-component-development]

Actions

Zoom - Get User's Call Logs https://developers.zoom.us/docs/zoom-phone/apis/#operation/phoneUserCallLogs

Zoom Admin - Get Account Call Logs https://developers.zoom.us/docs/zoom-phone/apis/#operation/accountCallHistory

[sergio-eliot-rodriguez]

Some notes while triagging:

User put his component request filed under sources, but the description: "I want to get all the call logs" suggest more of an action than a source. Can we confirm with the user?

Here is the relevant documentation for the related API endpoints: Get User's Call Logs Scopes required for this endpoint are: phone:read:admin, phone_call_log:read:admin, phone:read, phone_call_log:read

As per https://marketplace.zoom.us/apps/jGaV-kRrT3igAYnn-J5v2g the standard Pipedream app for Zoom is enough for Get User's Call Logs.

I checked on Zone Phone webhooks, from those related to call logs; I don't think those match the requested by the user: https://developers.zoom.us/docs/zoom-phone/webhooks/#operation/phone.callee_call_log_completed

[sergio-eliot-rodriguez]

Finally, I filed this under the Zoom app,

In case the user meant Get Account's Call Logs, and this actions needs to be implemented under the Zoom Admin app because the required scope for getting account's call logs are: phone:read:admin,phone_call_log:read:admin,

[vunguyenhung]

Hey @luancazarine, in the PR, I see the code for foursquare also got pushed. As they are unrelated, could you remove them from the PR code?

[vunguyenhung]

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information https://vunguyenhung.notion.site/ACTION-Zoom-app-Zoom-Phone-s-Get-Call-Logs-Action-ed13cc41202d4273bfabf4a50caca420

[luancazarine]

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information https://vunguyenhung.notion.site/ACTION-Zoom-app-Zoom-Phone-s-Get-Call-Logs-Action-ed13cc41202d4273bfabf4a50caca420

Hi @vunguyenhung, Those errors indicate that only accounts with Zoom Phone enabled can use these endpoints, so it's not a component error.

[vunguyenhung]

Yeah @luancazarine, I put changes required because it had foursquare code in the pr

[vunguyenhung]

Hello everyone, I have tested this PR and there're some test cases failed or needed improvement.

Please check the test report below for more information https://vunguyenhung.notion.site/ACTION-Zoom-app-Zoom-Phone-s-Get-Call-Logs-Action-ed13cc41202d4273bfabf4a50caca420

[vunguyenhung]

Hey @luancazarine, in the PR, I see the code for foursquare also got pushed. As they are unrelated, could you remove them from the PR code?

[luancazarine]

Hi @vunguyenhung Sorry, I hadn,t seen that, thanks for letting me know.

[vunguyenhung]

Hi everyone, all test cases are passed! Ready for release!

Test report https://vunguyenhung.notion.site/ACTION-Zoom-app-Zoom-Phone-s-Get-Call-Logs-Action-ed13cc41202d4273bfabf4a50caca420