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
feat: Revamp basic profile, avatar and message signature #9310
Conversation
muhsin-k
commented
Apr 27, 2024
Light mode | Dark mode |
---|---|
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.
Suggested a few changes. Also, what happens to the old page for profile settings?
app/javascript/dashboard/routes/dashboard/settings/personal/BasePersonalItem.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/BasePersonalItem.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/Index.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/Index.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/Index.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/MessageSignature.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/MessageSignature.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/UserBasicDetails.vue
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/UserBasicDetails.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/UserProfilePicture.vue
Outdated
Show resolved
Hide resolved
@scmmishra This just a temporary page for testing. Once I merge all the components for profile section. I will rename this route and delete the old profile page. |
app/javascript/dashboard/routes/dashboard/settings/personal/MessageSignature.vue
Outdated
Show resolved
Hide resolved
app/javascript/dashboard/routes/dashboard/settings/personal/personal.routes.js
Outdated
Show resolved
Hide resolved
…ssageSignature.vue Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com>
…rsonal.routes.js Co-authored-by: Sivin Varghese <64252451+iamsivin@users.noreply.github.com>
profileUpdate in auth.js handles `displayName` specifically, so adding the property separately as `display_name` caused the value to be overriden as empty
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.
Let's rewrite this in the setup syntax once the composables are merged. Approving it for now