Skip to content

Conversation

@nuno-vieira
Copy link
Member

@nuno-vieira nuno-vieira commented Nov 12, 2025

🔗 Issue Links

Related: https://github.com/GetStream/docs-content/pull/746

🎯 Goal

Remove timestamps from the demo app message reads info view, as per the documentation. Rendering the timestamp is useless since it is always the same for every message.

🧪 Manual Testing Notes

N/A.

☑️ Contributor Checklist

  • I have signed the Stream CLA (required)
  • This change should be manually QAed
  • Changelog is updated with client-facing changes
  • Changelog is updated with new localization keys
  • New code is covered by unit tests
  • Documentation has been updated in the docs-content repo

Summary by CodeRabbit

  • Refactor
    • Removed timestamp information from message delivery and read status indicators. Message status displays (delivered/read) remain fully functional and continue to show as expected.

@nuno-vieira nuno-vieira requested a review from a team as a code owner November 12, 2025 17:32
@nuno-vieira nuno-vieira added the 🪧 Demo App An Issue or PR related to the Demo App label Nov 12, 2025
@coderabbitai
Copy link

coderabbitai bot commented Nov 12, 2025

Walkthrough

The pull request removes timestamp-related functionality from the UserReadInfoRow component in the StreamChat demo app. This includes eliminating the timestamp parameter from the initializer, removing the timestamp property, and deleting associated formatting logic while preserving status rendering.

Changes

Cohort / File(s) Summary
UserReadInfoRow timestamp removal
DemoApp/StreamChat/Components/DeliveredMessages/DemoMessageReadsInfoView.swift
Removed timestamp parameter from initializer, eliminated timestamp property, deleted formatTimestamp() function, updated all call sites to pass only status (delivered/read)

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~10 minutes

  • Verify all call sites have been updated to remove timestamp arguments
  • Confirm timestamp property removal doesn't break other components or tests
  • Check that formatTimestamp deletion doesn't leave orphaned code

Suggested reviewers

  • martinmitrevski

Poem

🐰 A timestamp once adorned the reads,
But simplicity's what now succeeds!
Status shines bright without the date,
UserReadInfoRow's in better state! ⏰✨

Pre-merge checks and finishing touches

✅ Passed checks (3 passed)
Check name Status Explanation
Title check ✅ Passed The title accurately describes the main change: removing timestamps from the demo app's message reads info view component.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Description Check ✅ Passed Check skipped - CodeRabbit’s high-level summary is enabled.
✨ Finishing touches
  • 📝 Generate docstrings
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Post copyable unit tests in a comment
  • Commit unit tests in branch demo-app/remove-timestamps-from-message-read-info-view

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands and usage tips.

@nuno-vieira nuno-vieira merged commit 1f9b6c3 into develop Nov 13, 2025
14 checks passed
@nuno-vieira nuno-vieira deleted the demo-app/remove-timestamps-from-message-read-info-view branch November 13, 2025 16:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

🪧 Demo App An Issue or PR related to the Demo App

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants