Frun1na opened a new pull request, #5295:
URL: https://github.com/apache/rocketmq-dashboard/pull/5295
### Which Issue(s) This PR Fixes
- Fixes #5294
### Brief Description
The message table and the DLQ group table are paginated server-side (total
from the API), but four columns carried antd in-memory `sorter` functions.
Clicking a header reordered only the ~20-50 rows of the current page while
pages 2+ kept the server's original order, so the combined 'sorted' view was
silently wrong. The sorters are removed; ordering stays with the server, like
every other server-paginated table in the project.
### How Did You Test This Change?
```
cd web && npx vitest run src/pages/instance/__tests__/MessagePage.test.tsx
src/pages/instance/__tests__/DLQPage.test.tsx
Test Files 2 passed (2)
Tests 35 passed (35)
```
---
Re-submission of #4934: the original PR was closed by the stale bot after 7
days without review activity, and GitHub now rejects reopening pull requests in
this repository ("state cannot be changed"), so the same change is re-submitted
rebased onto the current `rocketmq-studio` tip (`a460673f`). The original
review thread stays available at #4934.
--
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.
To unsubscribe, e-mail: [email protected]
For queries about this service, please contact Infrastructure at:
[email protected]
