Skip to content

Auth makeover I guess - #7

Open
KallMeMoh wants to merge 15 commits into
mainfrom
auth-makeover-I-guess
Open

Auth makeover I guess#7
KallMeMoh wants to merge 15 commits into
mainfrom
auth-makeover-I-guess

Conversation

@KallMeMoh

Copy link
Copy Markdown
Contributor

Adds session provider for client components (might remove it icl)
Adds onboarding page (not tested)
Adds check inbox page
Adds activation route
Adds activation request page
Makes registration and login share a layout for now, old registration was hard to look at
Note: some parts don't fully work due to API inconsistency with the docs.

@KallMeMoh
KallMeMoh requested a review from kipteamm August 23, 2026 15:55
@KallMeMoh KallMeMoh self-assigned this Aug 23, 2026

@kipteamm kipteamm left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I can approve changes and you do the feature on another branch, or you do it on this one. eitherway fine by me

Comment thread src/app/activate/page.tsx

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Once they request to send a new one, it should be on cooldown for 1 minute before being able to retrigger. this may be a client-side cooldown because there is an actual backend ratelimit. smartasses 'bypassing' the clientside cooldown will just be ratelimited. we'd do it to improve regular user experience

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants