janpeter/scrum4me-workers
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity Actions

feat(messages): add result count filters #85

Merged
janpeter merged 1 commit from codex/fix-queue-message-table-layout into main 2026-08-19 16:58:23 +02:00
Conversation 1 Commits 1 Files changed 7 +181 -16
janpeter commented 2026-08-19 16:53:12 +02:00
Owner
Copy link

Samenvatting

  • voeg direct links van Reset de filters Laatste 25, 100 en Alles toe
  • behoud 100 als standaard en laat Reset daarnaar terugkeren
  • combineer de limiet met bestaande filters, inclusief vastzittende berichten
  • laat Alles de SQL LIMIT volledig verwijderen

Verificatie

  • npm run verify (879 tests geslaagd, 12 overgeslagen)
  • npm run build
  • onafhankelijke review: geen bevindingen
## Samenvatting - voeg direct links van Reset de filters Laatste 25, 100 en Alles toe - behoud 100 als standaard en laat Reset daarnaar terugkeren - combineer de limiet met bestaande filters, inclusief vastzittende berichten - laat Alles de SQL LIMIT volledig verwijderen ## Verificatie - npm run verify (879 tests geslaagd, 12 overgeslagen) - npm run build - onafhankelijke review: geen bevindingen
feat(messages): add result count filters
All checks were successful
CI / Verify (pull_request) Successful in 2m13s
Details
2f301715cf
s4m-codex-reviewer approved these changes 2026-08-19 16:55:57 +02:00
s4m-codex-reviewer left a comment
Copy link

APPROVED

Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden.

Findings

  • Geen blokkerende of error-severity findings gevonden.

De wijziging is consistent met de bestaande /queue/messages-patronen: default blijft 100, de UI-keuze wordt meegenomen in refreshes en reset, de server action vertaalt all bewust naar geen SQL LIMIT, en de relevante paden zijn met gerichte tests afgedekt.

# APPROVED Geen gekoppeld plan gevonden — beoordeeld op codekwaliteit + product-standaarden. ## Findings - Geen blokkerende of error-severity findings gevonden. De wijziging is consistent met de bestaande `/queue/messages`-patronen: default blijft 100, de UI-keuze wordt meegenomen in refreshes en reset, de server action vertaalt `all` bewust naar geen SQL `LIMIT`, en de relevante paden zijn met gerichte tests afgedekt.
Sign in to join this conversation.
Reviewers
No reviewers
s4m-codex-reviewer
Labels
Clear labels
No items
No labels
Milestone
Clear milestone
No items
No milestone
Projects
Clear projects
No items
No project
Assignees
Clear assignees
No assignees
2 participants
Notifications
Due date
The due date is invalid or out of range. Please use the format "yyyy-mm-dd".

No due date set.

Dependencies

No dependencies set.

Reference
janpeter/scrum4me-workers!85
No description provided.