-
Notifications
You must be signed in to change notification settings - Fork 497
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 Waitlist guide; add setup instructions to waitlist component reference #1734
Conversation
Hey, here’s your docs preview: https://clerk.com/docs/pr/1734 |
f214b6f
to
f67d3c8
Compare
Before using the `<Waitlist />` component, you must enable **Waitlist** mode in the Clerk Dashboard: | ||
|
||
1. In the Clerk Dashboard, navigate to the [**Restrictions**](https://dashboard.clerk.com/last-active?path=user-authentication/restrictions) page. | ||
1. Under the **Sign-up modes** section, enable **Waitlist**. |
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.
nit: This action should be Select
since it's a radio button.
4b45623
to
9ec0d5a
Compare
docs/guides/waitlist.mdx
Outdated
|
||
export default function RootLayout({ children }: { children: React.ReactNode }) { | ||
return ( | ||
<ClerkProvider waitlistUrl="/waitlist"> |
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.
@alexisintech I saw that you render the <Waitlist />
component in /
, so the waitlistUrl
prop should be /
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.
fixed! a93f692
docs/guides/waitlist.mdx
Outdated
import { SignIn } from '@clerk/nextjs' | ||
|
||
export default function Page() { | ||
return <SignIn waitlistUrl="/" /> |
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.
If we pass the waitlistUrl
in <ClerkProvider>
we don't need to add it here
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.
addressed: a93f692
blocked until redirects are merged: https://github.com/clerk/clerk/pull/811 |
Important
🔎 Previews:
Explanation:
This PR:
/docs/components/waitlist/overview
-->/docs/components/waitlist
Checklist