Skip to content

feat(tickets): add resolved date column - #5216

Merged
wescopeland merged 2 commits into
RetroAchievements:masterfrom
wescopeland:tickets-resolved-col
Sep 10, 2026
Merged

feat(tickets): add resolved date column#5216
wescopeland merged 2 commits into
RetroAchievements:masterfrom
wescopeland:tickets-resolved-col

Conversation

@wescopeland

Copy link
Copy Markdown
Member

It's possible to sort by resolved date in the new tickets list, but the field itself is missing a column definition. This PR adds the column definition, and automatically toggles column visibility when the sort is active.

Screenshot 2026-09-10 at 8 57 04 AM

@wescopeland
wescopeland requested a review from a team September 10, 2026 12:57
@greptile-apps

greptile-apps Bot commented Sep 10, 2026

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 5/5

The PR appears safe to merge.

Summary

  • Generalizes date-column rendering.
  • Preserves persisted visibility preferences.
  • Covers list and inbox behavior with tests.

Reviews (1) · Last reviewed commit: "feat(tickets): add resolved date column"

Jamiras
Jamiras approved these changes Sep 10, 2026
]);
if (section.kind === 'resolvedByYou') {
visibleColumnIds.add('resolvedAt');
}

@Jamiras Jamiras Sep 10, 2026

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Should this be generally available? The Manage > Tickets list can be filtered to resolved tickets.

Odd. Refreshing the page didn't add the column to the picker, but leaving and coming back did.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Things like this can sometimes happen locally if the Vite build cache is stale.

Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Double-checked locally to be extra sure, and confirmed things seem fine on my end.

@wescopeland
wescopeland merged commit 8e4c668 into RetroAchievements:master Sep 10, 2026
23 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.

2 participants