messere1 opened a new pull request, #1586: URL: https://github.com/apache/rocketmq-dashboard/pull/1586
## Summary - defer Clients instance loading state changes to a cancellable scheduled effect task - defer Consumer instance-scoped modal/list resets while preserving request-generation invalidation - defer DLQ instance-scoped cleanup and loading through the same cancellable lifecycle - suppress the Consumer table spinner when no instance is available ## Root cause The pages synchronously called React state setters from effect bodies. The current React hooks lint rules reject that pattern with five `react-hooks/set-state-in-effect` errors, so `npm run lint` failed on `rocketmq-studio`. ## User and developer impact The frontend lint gate passes again. Existing cancellation and stale-response guards remain intact, and the no-instance Consumer state no longer briefly exposes a loading spinner. Closes #1585 ## Validation - `npm run lint` (0 errors; existing non-blocking warnings remain) - `npm test -- --run src/pages/cluster/__tests__/ClientsPage.test.tsx src/pages/instance/__tests__/ConsumerPage.test.tsx src/pages/instance/__tests__/DLQPage.test.tsx` (30 tests passed) - `npm run build` -- 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]
