Skip to content

refactor: user avatar color - #17753

Merged
alextran1502 merged 1 commit into
mainfrom
refactor/avatar-color
Apr 28, 2025
Merged

alextran1502 merged 1 commit into
mainfrom
refactor/avatar-color

Conversation

@jrasm91

@jrasm91 jrasm91 commented Apr 21, 2025 •

Copy link
Copy Markdown
Member

Move UserAvatarColor to a dedicated column on the users table.

This is a breaking change because the following endpoints have changed:

  • GET /users/me/preferences no longer returns avatar: { color: UserAvatarColor }
  • PUT /users/me/preferences no long accepts avatar: { color: UserAvatarColor }

Similar changes exist for the /admin/users/:id/preferences endpoints.

These endpoints have been replaced with:

  • GET /users/me (returns avatarColor: UserAvatarColor)
  • PUT /users/me (accepts avatarColor: UserAvatarColor)

Similar changes exist for the /admin/users/:id endpoints.

@jrasm91
jrasm91 force-pushed the refactor/avatar-color branch from 4f5adca to 60d8e97 Compare April 21, 2025 16:58
@jrasm91 jrasm91 changed the title feat(mobile): map improvements (#17714) refactor: user avatar color Apr 21, 2025
@jrasm91
jrasm91 force-pushed the refactor/avatar-color branch from 60d8e97 to c08652d Compare April 21, 2025 18:38
@jrasm91
jrasm91 force-pushed the refactor/avatar-color branch from c08652d to 88a1e5b Compare April 24, 2025 21:02
@jrasm91
jrasm91 marked this pull request as ready for review April 24, 2025 21:23
@jrasm91
jrasm91 requested a review from danieldietzler as a code owner April 24, 2025 21:23

@danieldietzler danieldietzler 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

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants