Skip to content
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.

Sign up for GitHub

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

Jump to bottom

add: user dropdown #104

Merged
merged 9 commits into from
Oct 14, 2024
Merged

add: user dropdown #104

merged 9 commits into from
Oct 14, 2024

Conversation

kodaline
Copy link
Contributor

@kodaline kodaline commented Oct 9, 2024

Description

Added a user dropdown to manage logout, theme change

Current logged username is displayed as additional info

refers to issue #93

Type of change

Please delete options that are not relevant.

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • This change requires a documentation update

Checklist:

  • My code follows the style guidelines of this project
  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • My changes generate no new warnings

@kodaline kodaline marked this pull request as draft October 9, 2024 22:24
zAlweNy26
zAlweNy26 requested changes Oct 9, 2024
View reviewed changes
src/stores/useMainStore.ts Outdated Show resolved Hide resolved
src/stores/useMainStore.ts Outdated Show resolved Hide resolved
src/components/ThemeButton.vue Show resolved Hide resolved
src/components/Logout.vue Outdated Show resolved Hide resolved
@zAlweNy26
Copy link
Member

can you please rebase from develop?

kodaline and others added 9 commits October 14, 2024 15:23
@kodaline
add: user dropdown
ebb41df 
Added a user dropdown to manage logout, theme change

Current logged username is displayed as additional info

refers to issue cheshire-cat-ai#93
@kodaline
imp: user button styling change
e28cee3
@kodaline
imp: moved components
3210344 
Component ThemeButton and Logout moved inside UserDropdown
@kodaline
imp: useCookies
3247450
@kodaline
edit: redirect
3bf0c25
@kodaline
fix: text aligned to the right
de6a474
@kodaline
fix: signed in as area
f4eccb3
@zAlweNy26 @kodaline
small tweaks
d2dea0a
@zAlweNy26 @kodaline
Update components.d.ts
bf7430d
@kodaline kodaline marked this pull request as ready for review October 14, 2024 13:24
@zAlweNy26 zAlweNy26 linked an issue Oct 14, 2024 that may be closed by this pull request
Logout button #93
Closed
@zAlweNy26 zAlweNy26 merged commit a28a992 into cheshire-cat-ai:develop Oct 14, 2024
1 of 2 checks passed
@pieroit
Copy link
Member

pieroit commented Oct 14, 2024

AWESOOOOME

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@zAlweNy26 zAlweNy26 zAlweNy26 approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Logout button
3 participants
@kodaline @zAlweNy26 @pieroit