[client] Remove logout functionality from profile management #4336
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR removes logout functionality from the client's profile management system. The change streamlines the user interface by eliminating deregistration capabilities while maintaining core profile selection and removal features.
- Removed "Deregister" button from profile list UI
- Eliminated
handleProfileLogoutfunction and related confirmation dialogs - Removed logout menu item from system tray profile menu
Tip: Customize your code reviews with copilot-instructions.md. Create the file or learn how to get started.
You can also share your feedback on Copilot code review for a chance to win a $100 gift card. Take the survey.
| @@ -40,13 +40,12 @@ func (s *serviceClient) showProfilesUI() { | |||
| list := widget.NewList( | |||
| func() int { return len(profiles) }, | |||
| func() fyne.CanvasObject { | |||
| // Each item: Selected indicator, Name, spacer, Select, Logout & Remove buttons | |||
| // Each item: Selected indicator, Name, spacer, Select & Remove buttons | |||
There was a problem hiding this comment.
[nitpick] The comment describes the UI layout but uses '&' instead of 'and' which is inconsistent with typical code comment style. Consider using 'and' for better readability.
| // Each item: Selected indicator, Name, spacer, Select & Remove buttons | |
| // Each item: Selected indicator, Name, spacer, Select and Remove buttons |
Copilot uses AI. Check for mistakes.
|
Describe your changes
[client] Remove logout functionality from profile management
Issue ticket number and link
Stack
Checklist
Documentation
Select exactly one:
Docs PR URL (required if "docs added" is checked)
Paste the PR link from https://github.com/netbirdio/docs here:
https://github.com/netbirdio/docs/pull/__