fix: list Slack as an auth provider for the authorization URL#633
fix: list Slack as an auth provider for the authorization URL#633gjtorikian merged 2 commits intoworkos:mainfrom
Conversation
Greptile SummaryThis PR expands the Confidence Score: 5/5Safe to merge — no P0/P1 issues; only a P2 note about manual edits to auto-generated files. All Enum/Literal pairs are in sync, the previous P1 is resolved, and no logic or security concerns are introduced. The sole open concern (auto-generated file header) is P2. No files require special attention beyond the auto-generated file caveat already noted. Important Files Changed
Flowchart%%{init: {'theme': 'neutral'}}%%
flowchart TD
A[get_authorization_url] --> B{provider type?}
B --> C[UserManagementAuthenticationProvider Enum]
B --> D[UserManagementAuthenticationProviderLiteral TypeAlias]
C --> E[authkit / AppleOAuth / BitbucketOAuth / DiscordOAuth / GitHubOAuth / GitLabOAuth / GoogleOAuth / IntuitOAuth / LinkedInOAuth / MicrosoftOAuth / SalesforceOAuth / SlackOAuth NEW / VercelMarketplaceOAuth / VercelOAuth / XeroOAuth]
D --> E
F[SSOProvider Enum] --> G[AppleOAuth / BitbucketOAuth / DiscordOAuth / GitHubOAuth / GitLabOAuth / GoogleOAuth / IntuitOAuth / LinkedInOAuth / MicrosoftOAuth / SalesforceOAuth / SlackOAuth NEW / VercelMarketplaceOAuth / VercelOAuth / XeroOAuth]
H[SSOProviderLiteral TypeAlias] --> G
Reviews (3): Last reviewed commit: "update with full list" | Re-trigger Greptile |
04eaf1b to
965d07e
Compare
|
Hey, thanks for this! These files are now automatically generated by our OpenAPI spec and completely machine generated ( |
Description
Slack wasn't in here, but it works. I want to use it in
workos_client.user_management.get_authorization_url. (This list seems somewhat inconsistent with other lists).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.