-
Notifications
You must be signed in to change notification settings - Fork 11
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
Add organization membership deactivate and reactivate API methods #264
Add organization membership deactivate and reactivate API methods #264
Conversation
@@ -300,6 +307,7 @@ def list_organization_memberships( | |||
Kwargs: | |||
user_id (str): Filter Organization Memberships by user. (Optional) | |||
organization_id (str): Filter Organization Memberships by organization. (Optional) | |||
statuses (list): Filter Organization Memberships by status. (Optional) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
do we generally use this naming pattern for list params?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Do mean the pluralized parameter name? Seems like we do in the Python SDK:
https://github.com/workos/workos-python/blob/main/workos/organizations.py#L41
https://github.com/workos/workos-python/blob/main/workos/events.py#L38
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
no strong opinion, but statuses
seems a bit weird. Maybe status_list
?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Down to change it. That being said statuses
is how it's defined in the API/other SDKs. Maybe it's better to be consistent, thoughts?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I think we should be consistent and stick with statuses
.
…on-membership-method-to-python-sdk
Description
Add organization membership deactivate and reactivate API methods.
Documentation
Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.
If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.