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

Add Apple to list of providers. #323

Merged
merged 4 commits into from
Aug 9, 2024
Merged

Conversation

MikeMcQuaid
Copy link
Contributor

This may be me completely understanding this code but: this looked like an omission here so I thought I'd open a PR for discussion.

Description

Documentation

Does this require changes to the WorkOS Docs? E.g. the API Reference or code snippets need updates.

[ ] Yes

If yes, link a related docs PR and add a docs maintainer as a reviewer. Their approval is required.

This may be me completely understanding this code but: this looked like an omission here so I thought I'd open a PR for discussion.
@MikeMcQuaid MikeMcQuaid changed the title user_management: add Apple to list of providers. Add Apple to list of providers. Aug 8, 2024
Copy link
Contributor

@mthadley mthadley left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

That was not an intentional omission. Thanks for fixing, @MikeMcQuaid!

@mthadley mthadley merged commit ef7c82d into workos:main Aug 9, 2024
4 checks passed
@MikeMcQuaid MikeMcQuaid deleted the patch-1 branch August 9, 2024 06:35
@mthadley mthadley mentioned this pull request Aug 9, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

2 participants