fix: logout button not working in sidebar

[pratyushjha06]

Pull Requests Review Criteria

Caution

PRs that fail to meet these review standards will be automatically flagged and may be rejected by maintainers.

• Filled out this PR template properly
• Did not commit directly to main
• Limited commits (max 3–4 unless permitted by Admin/Mentors)
• Added comments in complex parts of code

Closes: #1357

Describe the add-ons or changes you've made 📃

1. Uncommented and implemented onLogout function in useSidebar hook
2. Integrated with existing useAuth hook for proper state management
3. Fixed handleClick logic in SidebarMenuItem to check onClick first before path
4. Added redirect to /login after logout

Changes

• shared/components/organisms/sidebar/hooks/index.ts - Added onLogout using useAuth
• shared/components/organisms/sidebar/components/SidebarMenuItem.tsx - Fixed onClick

Screenshots 📷

16-41-00.-.Trim.mp4

Note to reviewers (Optional) 📄

[vercel]

@pratyushjha06 is attempting to deploy a commit to the avdheshvarshney's projects Team on Vercel.

A member of the Team first needs to authorize it.

[github-actions]

Thank you for submitting your pull request! We'll review it as soon as possible. For further communication, join our discord server https://discord.gg/tSqtvHUJzE.

[Avdhesh-Varshney]

No need to write a separate function for logout.

Just pass the logout to onClick parameter.

[Avdhesh-Varshney]

Revert the changes of this file. These are not required.

[pratyushjha06]

Revert the changes of this file. These are not required.

Done

[vercel]

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

Project	Deployment	Review	Updated (UTC)
code-a2z	Ready	Preview, Comment	Jan 25, 2026 9:36am

[Avdhesh-Varshney]

Hey @pratyushjha06, Why have you added new lines in file? (This breaks formatting)

Great work. PR Approved 🎉