-
Notifications
You must be signed in to change notification settings - Fork 12
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 async support for organizations #334
Add async support for organizations #334
Conversation
@@ -1,11 +1,9 @@ | |||
from typing import Generic, Literal, Union |
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.
Cleaning up unused imports.
48aa634
to
5231694
Compare
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.
|
||
|
||
@pytest.mark.asyncio | ||
class TestAsyncOrganizations(OrganizationFixtures): |
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.
This repetition is so grrrrr.
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 know 😢 We need some more thought to strike the right balance between little repetition and and readability.
@@ -75,29 +78,39 @@ def create_organization( | |||
name: str, | |||
domain_data: Optional[Sequence[DomainDataInput]] = None, | |||
idempotency_key: Optional[str] = None, | |||
) -> Organization: ... | |||
) -> SyncOrAsync[Organization]: | |||
"""Create an organization |
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.
Odd. I wonder why this is in the diff Is it actually missing?
Description
Add async support for organizations. This is the last module I felt like async support could absolutely be useful, in tandem with other modules.
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.