-
Notifications
You must be signed in to change notification settings - Fork 1
feat: profile preferences #168
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This pull request introduces user profile preferences functionality, including a new preferences page for managing email digest subscriptions and a profile dropdown menu in the application header.
Changes:
- Added a new PreferencesView page allowing users to manage email notification settings and view account information
- Introduced a ProfileDropdown component in the application header with user info display and logout functionality
- Restructured the main application layout to include a sticky header with the profile dropdown
- Added eslint-disable comments for type assertions across catalog-related components
Reviewed changes
Copilot reviewed 14 out of 14 changed files in this pull request and generated 9 comments.
Show a summary per file
| File | Description |
|---|---|
| src/views/PreferencesView.vue | New page for user preferences with email digest subscription toggle and account info display |
| src/components/ProfileDropdown.vue | New profile dropdown component with user avatar, preferences link, and logout button |
| src/views/layouts/App.vue | Restructured layout to add header with profile dropdown; moved padding from main to content wrapper |
| src/router/index.ts | Added new /preferences route requiring authentication |
| src/views/catalog/CatalogView.vue | Removed unused parameters and imports; added eslint comment for type assertion |
| src/views/catalog/CatalogListView.vue | Removed unused API execute function |
| src/views/catalog/CatalogCreateView.vue | Added eslint-disable comment for type assertion |
| src/components/catalogs/*.vue | Added eslint-disable comments for type assertions in transformRequest callbacks |
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 14 out of 14 changed files in this pull request and generated 4 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 16 out of 16 changed files in this pull request and generated 3 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
Signed-off-by: Gustavo Carvalho <gustavo.carvalho@container-solutions.com>
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
Copilot reviewed 16 out of 16 changed files in this pull request and generated 7 comments.
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
No description provided.