Skip to content

Conversation

@erikeldridge
Copy link
Contributor

Internal client updates.

This is based on PR #2457, which updated the API client.

Discussion

Working with @lahirumaramba and @trekforever.

Testing

Ran npm test and all tests pass.

@erikeldridge erikeldridge self-assigned this Feb 14, 2024
@erikeldridge erikeldridge requested a review from jenh February 14, 2024 20:04
Base automatically changed from ssrc-client-internals to ssrc February 20, 2024 16:51
@erikeldridge erikeldridge requested a review from jenh February 20, 2024 18:01
@erikeldridge
Copy link
Contributor Author

Thank you for reviewing, @jenh. I've updated this PR with your suggested changes.

Co-authored-by: jen_h <harveyjen@google.com>
Copy link
Member

@lahirumaramba lahirumaramba left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Discussed the logging offline. LGTM!

@erikeldridge erikeldridge merged commit 5c9b649 into ssrc Mar 5, 2024
@erikeldridge erikeldridge deleted the ssrc-rc branch March 5, 2024 22:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

4 participants