-
Notifications
You must be signed in to change notification settings - Fork 14
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
GetConnection(s) to Support Pin Groups #639
Conversation
As this PR has a dependency on the previous one PR-636, I've raised it on top of that. I have locally tested these changes, and all the tests are passing. Kindly review this PR. I will retarget it to the main branch once the other PR is approved and merged. Cc: @dixonjoel, @bkeryan |
Test Results 25 files 25 suites 31m 1s ⏱️ Results for commit 1d3f502. ♻️ This comment has been updated with latest results. |
What does this Pull Request accomplish?
_get_connections_core
API.Why should this Pull Request be merged?
What testing has been done?
Existing and new tests passes.