Skip to content

Dev to opportunities - #2153

Merged
jmgasper merged 6 commits into
opportunities-v6from
dev
Aug 14, 2026
Merged

Dev to opportunities#2153
jmgasper merged 6 commits into
opportunities-v6from
dev

Conversation

@jmgasper

Copy link
Copy Markdown
Collaborator

Related JIRA Ticket:

https://topcoder.atlassian.net/browse/

What's in this PR?

What was broken

After assigning a ticket from the Support ticket list, the detail screen could still show the ticket as unassigned and offer "Assign to me" until the page was refreshed.

Root cause

The asynchronous mark-read completion performed a data-bearing SWR mutation that could supersede an in-flight detail revalidation and preserve stale pre-assignment data.

What was changed

Changed the post-read cache update to a revalidation-only SWR mutation, ensuring the detail page reloads current server state without invalidating a concurrent assignment refresh.

Any added/updated tests

Updated the TicketDetailPage regression test to verify that completing the mark-read request triggers a zero-argument, revalidation-only mutation.
What was broken
The View challenge action on the ticket details page was clickable but appeared as ordinary gray text.

Root cause
The global link reset removes native color and decoration, while the Support app did not define the CSS variable used by this link.

What was changed
Styled the challenge link and its hover state with the shared accessible blue token and a persistent underline.

Any added/updated tests
Added a focused stylesheet regression assertion covering the challenge link's default and hover presentation. The existing semantic link and encoded URL assertions remain in place.
PM-5848: Refresh support ticket details after read
PM-5854: Make challenge link visually distinct
@jmgasper
jmgasper requested a review from kkartunov as a code owner August 14, 2026 06:03
@jmgasper
jmgasper merged commit a0bbc65 into opportunities-v6 Aug 14, 2026
5 of 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