Skip to content

feat: add log out button to sidebar header menu#603

Merged
NagariaHussain merged 1 commit intofrappe:developfrom
NagariaHussain:feat/logout-button
Apr 22, 2026
Merged

feat: add log out button to sidebar header menu#603
NagariaHussain merged 1 commit intofrappe:developfrom
NagariaHussain:feat/logout-button

Conversation

@NagariaHussain
Copy link
Copy Markdown
Collaborator

@NagariaHussain NagariaHussain commented Apr 22, 2026

Closes #600

image

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 22, 2026

Walkthrough

The Sidebar component was updated to add a logout feature. The menu header configuration now includes two items instead of one: the existing "Toggle Theme" option and a new "Log out" action. The implementation adds imports for the logout icon and session store, introduces a logout handler that invokes the session store's logout method, and wires this handler to the menu item's click event.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: adding a logout button to the sidebar header menu, which matches the changeset modifications in Sidebar.vue.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.

✏️ Tip: You can configure your own custom pre-merge checks in the settings.


Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@NagariaHussain NagariaHussain merged commit a09ef22 into frappe:develop Apr 22, 2026
4 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.

How to log out of Wiki

1 participant