Skip to content
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

Added support for "include_totals" to all_organization_member_roles #635

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

jpayton-cx
Copy link

@jpayton-cx jpayton-cx commented Dec 17, 2024

Changes

Please describe both what is changing and why this is important. Include:

  • the get-org-member-roles is supposed to support an include_totals query param in addition to page and per_page, but the SDK does not expose it. Added support for the include_totals parameter to the all_organization_member_roles function in the Organizations class as outlined by the API documentation.

References

Please include relevant links supporting this change such as a:

Documentation for the get-organization-member-roles endpoint: https://auth0.com/docs/api/management/v2/organizations/get-organization-member-roles

Testing

Please describe how this can be tested by reviewers. Be specific about anything not tested and reasons why. If this library has unit and/or integration testing, tests should be added for new functionality and existing tests should complete without errors.

  • This change adds unit test coverage
  • This change adds integration test coverage
  • This change has been tested on the latest version of the platform/language or why not
    Tested using python 3.13

Tested locally on my machine and modified unit tests to reflect the changes. Verified that all tests pass with the updated code. also verified that the include_totals parameter is actually supported by the API

Checklist

@jpayton-cx jpayton-cx requested a review from a team as a code owner December 17, 2024 00:57
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.

1 participant