Skip to content

chore: UItest - read receipt on group conversation - WPB-25831#5010

Open
findms wants to merge 2 commits into
release/cycle-4.24from
chore/ms-test-change-read-receipt-setting-WPB-25831
Open

chore: UItest - read receipt on group conversation - WPB-25831#5010
findms wants to merge 2 commits into
release/cycle-4.24from
chore/ms-test-change-read-receipt-setting-WPB-25831

Conversation

@findms

@findms findms commented Jul 14, 2026

Copy link
Copy Markdown
Contributor
TaskWPB-25831 [iOS] Automation - As a user, I am able to change read receipt setting on group conversations

Issue

UiTest- read receipt enabling on group conversations

Testing

Describe how to test.

Optional: attachments like images, videos, etc.

readreceipts.mp4

Checklist

  • Title contains a reference JIRA issue number like [WPB-XXX].
  • Description is filled and free of optional paragraphs.
  • Adds/updates automated tests.

UI accessibility checklist

If your PR includes UI changes, please utilize this checklist:

  • Make sure you use the API for UI elements that support large fonts.
  • All colors are taken from WireDesign.ColorTheme or constructed using WireDesign.BaseColorPalette.
  • New UI elements have Accessibility strings for VoiceOver.

@findms
findms marked this pull request as ready for review July 14, 2026 13:58
@sonarqubecloud

Copy link
Copy Markdown

@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

Test Results

2 099 tests   2 071 ✅  3m 28s ⏱️
  345 suites     28 💤
    2 files        0 ❌

Results for commit 5e351c8.

♻️ This comment has been updated with latest results.

Summary: workflow run #29406466864
Allure report (download zip): html-report-32002-chore_ms-test-change-read-receipt-setting-WPB-25831

@findms
findms requested review from David-Henner and acv-w July 15, 2026 09:58
Comment on lines +538 to +539
? "You turned read receipts on for everyone"
: "You turned read receipts off for everyone"

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

suggestion: I feel like this is too brittle. Would it perhaps be possible to have different accessibility identifiers for the system message?

@findms
findms changed the base branch from develop to release/cycle-4.24 July 17, 2026 13:28
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.

2 participants