Conversation
…sponse The DashboardRevisionSerializer was returning only id, name, and email for the createdBy field. Include avatarUrl so callers can display the user's avatar without constructing it client-side from the email. Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
github-actions
Bot
added
the
Scope: Backend
gggritso
approved these changes
Apr 28, 2026
skaasten
added a commit
that referenced
this pull request
Apr 28, 2026
Wire the `avatarUrl` now returned by the revisions API into `UserAvatar` so the revision list shows the author's avatar instead of always falling back to initials. Depends on #114186. <img width="700" height="149" alt="Screenshot 2026-04-28 at 1 19 11 PM" src="https://github.com/user-attachments/assets/5812ded5-f530-49f6-bde2-b19124255dec" /> Refs #DAIN-1579 Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
skaasten
added a commit
that referenced
this pull request
Apr 30, 2026
…revision list (#114336) After #114186, we have the avatar type being returned in the revision list endpoint. We should pass that into the avatar component so that it properly shows initials vs upload vs gravatar. The current-version row in the modal passes a full `User` object whose avatar info is nested under `avatar.avatarType`. That field is now explicitly mapped to the flat `createdBy` prop shape so it follows the same rendering path as historical revision rows. --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Claude Sonnet 4.6 <noreply@example.com>
3 tasks
cleptric
pushed a commit
that referenced
this pull request
May 5, 2026
…sponse (#114186) Add the user avatar to the dashboard revision list serializer so it can be displayed by the front end (follow-up PR). I've tested locally that the front-end still works with this new attribute added. Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
cleptric
pushed a commit
that referenced
this pull request
May 5, 2026
Wire the `avatarUrl` now returned by the revisions API into `UserAvatar` so the revision list shows the author's avatar instead of always falling back to initials. Depends on #114186. <img width="700" height="149" alt="Screenshot 2026-04-28 at 1 19 11 PM" src="https://github.com/user-attachments/assets/5812ded5-f530-49f6-bde2-b19124255dec" /> Refs #DAIN-1579 Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com>
cleptric
pushed a commit
that referenced
this pull request
May 5, 2026
…revision list (#114336) After #114186, we have the avatar type being returned in the revision list endpoint. We should pass that into the avatar component so that it properly shows initials vs upload vs gravatar. The current-version row in the modal passes a full `User` object whose avatar info is nested under `avatar.avatarType`. That field is now explicitly mapped to the flat `createdBy` prop shape so it follows the same rendering path as historical revision rows. --------- Co-authored-by: Claude Sonnet 4.6 <noreply@anthropic.com> Co-authored-by: Claude Sonnet 4.6 <noreply@example.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Add the user avatar to the dashboard revision list serializer so it can be displayed by the front end (follow-up PR).
I've tested locally that the front-end still works with this new attribute added.