Skip to content

Refactor layout and onboarding components for improved clarity and fu…#515

Merged
LuizGarbini merged 2 commits intomainfrom
garbini/plo-256-onboarding-profile-layout-fix
Apr 24, 2026
Merged

Refactor layout and onboarding components for improved clarity and fu…#515
LuizGarbini merged 2 commits intomainfrom
garbini/plo-256-onboarding-profile-layout-fix

Conversation

@LuizGarbini
Copy link
Copy Markdown
Contributor

  • Removed unused headers and onboarding state from RootLayout, simplifying the component structure.
  • Updated call-to-action text in OnboardingCelebration to use a more relevant label.
  • Moved onboarding state determination to LayoutWrapper using usePathname for better encapsulation.

Describe your changes

Issue ticket number and link

Checklist before requesting a review

  • I have performed a self-review of my code
  • If it's an essential feature, I've tested it thoroughly.
  • Do we need to implement analytics?
  • Will this be part of a product update? If yes, please write one phrase about this update.

…nctionality

- Removed unused headers and onboarding state from RootLayout, simplifying the component structure.
- Updated call-to-action text in OnboardingCelebration to use a more relevant label.
- Moved onboarding state determination to LayoutWrapper using usePathname for better encapsulation.
@linear
Copy link
Copy Markdown

linear Bot commented Apr 18, 2026

@vercel
Copy link
Copy Markdown

vercel Bot commented Apr 18, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
plotwist Ready Ready Preview, Comment Apr 18, 2026 1:48am

Request Review

- Introduced a new function to extract and validate locale information from the redirect URL.
- Updated the sign-in function to utilize the extracted locale, enhancing user experience by ensuring correct language settings during authentication.
@LuizGarbini LuizGarbini merged commit de8cdeb into main Apr 24, 2026
3 checks passed
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.

1 participant