-
Notifications
You must be signed in to change notification settings - Fork 393
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
docs: added documentation for the patreon oauth provider #1211
Conversation
Co-authored-by: Tom Papiernik <[email protected]>
Apologies for not fixing this up sooner. I will make the requested changes tonight after work. |
@AUSBird that's okay! Please update the way you list claims, cut down on the number of scopes in the config snippet and use a plain link in the snippet instead of a hyperlink and we should be good to merge. |
@tomekpapiernik I have made the changes. I swapped the claim list into a list that doesn't add long descriptions to each claim (That said they are kind of obvious what they do) two claims have a slash cause they are the same. And I removed all but two scopes in the config example with a link to the docs with the rest of the scopes. |
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.
Thanks for your contribution! 🎉 I've applied minor cosmetic tweaks myself to speed things up. Merging once all tests pass. Cheers!
Cheers, thanks for the help :) |
Added documentation for a PR that adds a patreon OIDC provider
Related Issue or Design Document
PR for that added the
patreon
oidc provider to Kratosory/kratos#3021
There is no issue on the documentation repository
Checklist
If this pull request addresses a security vulnerability,
I confirm that I got approval (please contact [email protected]) from the maintainers to push the changes.
Further comments
None