Skip to content

PM-5857: Label support team replies - #2141

Merged
jmgasper merged 2 commits into
devfrom
PM-5857
Aug 14, 2026
Merged

PM-5857: Label support team replies#2141
jmgasper merged 2 commits into
devfrom
PM-5857

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

What was broken

Support staff and ticket raiser replies displayed only their handles, making their roles difficult to distinguish.

Root cause

The ticket conversation rendered reply handles without using the existing author and ticket-owner IDs to identify support-authored responses.

What was changed

Replies authored by someone other than the ticket owner now display {handle} (Support Team). Ticket-owner replies remain unchanged, including when the owner also holds the Support Team role.

Any added/updated tests

Added TicketDetailPage coverage for both ticket-owner and support-authored replies.

Validation passed:

  • Focused TicketDetailPage suite: 6 tests
  • Complete Support app suite: 9 suites, 30 tests
  • Lint
  • Production build (with existing warnings)

The complete monorepo test command was also run. It reported 222 passing suites and 18 failing suites in unchanged Work, Engagements, Wallet, and Customer Portal code due existing alias, mock, and expectation issues; the Support suites all passed.

What was broken
Support staff and ticket raiser replies displayed only their handles, making their roles difficult to distinguish.

Root cause
The ticket conversation rendered reply handles without using the existing author and ticket-owner IDs to identify support-authored responses.

What was changed
Append (Support Team) to replies authored by someone other than the ticket owner. Ticket-owner replies remain unchanged, including when the owner also holds the Support Team role.

Any added/updated tests
Added TicketDetailPage coverage that verifies the suffix appears for support replies and not for ticket-owner replies.
@jmgasper
jmgasper requested a review from kkartunov as a code owner August 12, 2026 10:59
Resolve the TicketDetailPage test conflict while preserving both reply-role labelling and the newer assignment/closure coverage. Fix stale deferred-screener fixture references exposed by the merged dev build.
@jmgasper
jmgasper merged commit 03b20d6 into dev Aug 14, 2026
7 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.

1 participant