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

[core] Add more OAuth providers to SignInPage #3933

Merged
merged 25 commits into from
Sep 12, 2024

Conversation

bharatkashyap
Copy link
Member

@bharatkashyap bharatkashyap commented Aug 14, 2024

@bharatkashyap bharatkashyap added the enhancement This is not a bug, nor a new feature label Aug 14, 2024
@bharatkashyap bharatkashyap changed the title [core] Add more OAuth providers [core] Add more OAuth providers to SignInPage Aug 16, 2024
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 20, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 22, 2024
- Discord
- Keycloak
- Credentials (username/password)
Find the list of supported providers in the [API reference](/toolpad/core/api/sign-in-page/#sign-in-page-prop-providers).
Copy link
Member

Choose a reason for hiding this comment

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

This feels worse. Can't we do the opposite? Keep the list here and just call it a string in the API docs?

@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Aug 29, 2024
@github-actions github-actions bot added PR: out-of-date The pull request has merge conflicts and can't be merged and removed PR: out-of-date The pull request has merge conflicts and can't be merged labels Aug 30, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 2, 2024
@github-actions github-actions bot added the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 5, 2024
@github-actions github-actions bot removed the PR: out-of-date The pull request has merge conflicts and can't be merged label Sep 12, 2024
@bharatkashyap
Copy link
Member Author

Merging with codeCov decreasing, this is a consequence of the icons being imported

@bharatkashyap bharatkashyap merged commit 6b7e22e into mui:master Sep 12, 2024
13 of 14 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement This is not a bug, nor a new feature
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add more OAuth Providers and update docs, CLI
3 participants