Add OAuthConsent and useOAuthConsent docs#3315
Conversation
|
The latest updates on your projects. Learn more about Vercel for GitHub.
|
|
@wobsoriano have submitted a docs review with some minor changes:
Let me know what you think, and left some comments too! |
|
this is perfect! I tested with typedoc generation, and everything is working. my only qualm is putting |
Yeah I was in the same boat yesterday debating whether it should be in |
@wobsoriano @alexisintech just pushed a commit to make OAuthApplication a type rather than an object. Let me know thoughts! |
|
|
||
| --- | ||
|
|
||
| - `oauthApplication` |
There was a problem hiding this comment.
so it does exist on the clerk object, so it's fine to keep it here. i just meant that it doesn't need be in /objects and under objects in the manifest!
There was a problem hiding this comment.
also, towards the bottom there's a section called "Components"
it will need a section for mountOAuthConsent and unmountOAuthconsent like the other components listed there!
There was a problem hiding this comment.
|
could you please add all the relevant links added to the PR description here? any JS repo PR's that were related to releasing this future, and the PR's that were related to typedoc changes |
@wobsoriano have added the Typedoc PRs related to this but I'll let you add the JS repo PR'S related to releasing this feature. |
|
Okay @wobsoriano @alexisintech this is ready for a final review! |
🔎 Previews:
What does this solve? What changed?
We will now allow C1s to configure a custom OAuth consent page where they can use a custom hook, or the new drop-in
<OAuthConsent />componentDeadline
Other resources