Skip to content

Show "You:" prefix for last message in conversation list #1323

Merged
tunjid merged 4 commits into
mainfrom
joel/conversation-last-message-you-prefix
May 22, 2026
Merged

Show "You:" prefix for last message in conversation list #1323
tunjid merged 4 commits into
mainfrom
joel/conversation-last-message-you-prefix

Conversation

@joelmuraguri

Copy link
Copy Markdown
Collaborator

Closes #1317

What changed:

  • Updated Conversation.summary() to accept signedInProfileId and compare against message.sender.did
  • When the signed-in user sent the last message, the summary now prefixes with You:
  • Added you_sent_prefix string resource
  • Threaded signedInProfileId from the Conversation composable down into the summary() call

@joelmuraguri joelmuraguri requested a review from tunjid May 21, 2026 19:09

@gemini-code-assist gemini-code-assist Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Code Review

This pull request adds a "You: " prefix to the conversation summary when the last message was sent by the signed-in user. The review feedback recommends using a format string instead of manual concatenation and trailing spaces in the string resource to improve localization support.

Comment thread feature/messages/src/commonMain/composeResources/values/strings.xml Outdated
@tunjid tunjid merged commit c9d32e2 into main May 22, 2026
5 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Show who sent the last message in MessagesScreen

2 participants