Skip to content

feat: adding settings to change name/email and sign-out confirmation#24

Merged
CPrutean merged 2 commits into
mainfrom
profile-editing
Jun 23, 2026
Merged

feat: adding settings to change name/email and sign-out confirmation#24
CPrutean merged 2 commits into
mainfrom
profile-editing

Conversation

@RadAlpaca11

@RadAlpaca11 RadAlpaca11 commented Jun 20, 2026

Copy link
Copy Markdown
Member

What changed

Settings:

  • added a settings page when clicking name on sidebar
  • added ability to change display name (updates right away)
  • added ability to change email (requires verification from new email)

Sign out confirmation

  • added a pop-up confirmation check to make sure you really want to sign out

Why

Settings:

There was no way to update your profile at all, and it's a good feature to have.

Sign out confirmation

  • helps prevent accidentally signing out

Type of change

  • Feature (feat/)
  • Bug fix (fix/)
  • Chore / infra (chore/)
  • Docs (docs/)
  • Refactor (refactor/)

Checklist

  • pnpm lint passes
  • pnpm format:check passes
  • If schema.ts was changed - migration generated (pnpm db:generate) and committed alongside the schema
  • PR is focused on one concern

@RadAlpaca11 RadAlpaca11 self-assigned this Jun 20, 2026
@RadAlpaca11 RadAlpaca11 added the enhancement New feature or request label Jun 20, 2026
@CPrutean

Copy link
Copy Markdown
Member

Looks good! i think one thing id like to potentially suggest is maybe setting a cooldown to the name changes, i dont want people going in spamming name changes and making things chaotic

@RadAlpaca11

Copy link
Copy Markdown
Member Author

Looks good! i think one thing id like to potentially suggest is maybe setting a cooldown to the name changes, i dont want people going in spamming name changes and making things chaotic

Added!

@CPrutean CPrutean left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@CPrutean CPrutean merged commit e63a2cc into main Jun 23, 2026
2 checks passed
@CPrutean CPrutean deleted the profile-editing branch June 23, 2026 01:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants