RockteMQ-AI commented on issue #4244: URL: https://github.com/apache/rocketmq-dashboard/issues/4244#issuecomment-5639367642
Confirmed. The create path at `consumer.tsx:2288` and the CSV import path at `consumer.tsx:824-829` both splice created groups into local state via `setGroups(...)` without refetching the server-paginated list or updating `totalGroups`. The delete paths in the same file already do this correctly via `reloadConsumerGroupPageAfterDelete`, so the fix pattern is established. Note: the topic page's CSV import (`topic.tsx:1220-1225`) has the identical defect — its single create was fixed to call `reloadTopicPage()` but the import path still does a local-only splice. The fix PR should ideally cover both pages' import paths to prevent this bug class from recurring. Suggested fix: replace the local `setGroups(...)` calls in both the single-create `onOk` and the import completion handler with `await loadConsumerGroupPage(page, pageSize)` (or `triggerRefresh()`), mirroring the delete paths. Same for the topic import path. -- 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]
