Skip to content

refactor: integrate AppLayout into user profile page and simplify Adm…#1838

Open
akinboyewaSamson wants to merge 1 commit into
EarnQuestOne:mainfrom
akinboyewaSamson:feat/theme-toggle-all-routes
Open

refactor: integrate AppLayout into user profile page and simplify Adm…#1838
akinboyewaSamson wants to merge 1 commit into
EarnQuestOne:mainfrom
akinboyewaSamson:feat/theme-toggle-all-routes

Conversation

@akinboyewaSamson

Copy link
Copy Markdown

closes #1673

Description

This PR addresses incomplete/broken layouts across administrative and profile routes by safely implementing the dark/light mode toggle in the AdminLayout and wiring up necessary context hooks to the user profile dashboard.

Changes Made

AdminLayout.tsx`:

  • Fixed broken layout file and complete structural tags.
  • Integrated the dynamic <ThemeToggle /> component into the top admin header.
  • Cleaned up the side navigation schema (Dashboard, Quests, Submissions, Users) to adapt seamlessly across zinc-50 light and zinc-900 dark modes.

Profile page.tsx:

  • Fixed incomplete file closure.
  • Correctly parsed the address dynamic slug param using Next.js useParams().
  • Connected user profile state variables (profileData, isLoading) and structured interaction callbacks (refetch, updateProfileData, follow, unfollow) down into the <UserProfile /> presentational component layout.

Checklist

  • Code compiles cleanly without shell syntax errors or dangling layout tokens.
  • Theme switching functions correctly inside administrative sub-routes.
  • State management logic maps perfectly down to the presentation tier of the profile view.
  • Branch history verified clean and ready for code review.

@drips-wave

drips-wave Bot commented Jun 29, 2026

Copy link
Copy Markdown

@akinboyewaSamson Great news! 🎉 Based on an automated assessment of this PR, the linked Wave issue(s) no longer count against your application limits.

You can now already apply to more issues while waiting for a review of this PR. Keep up the great work! 🚀

Learn more about application limits

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.

Implement Dark Mode Toggle Across All Inner Pages

2 participants