123123213weqw opened a new pull request, #1452:
URL: https://github.com/apache/rocketmq-dashboard/pull/1452

   ## What is the purpose of the change
   
   Fixes #1450.
   
   The Topic detail dialog's rebuild path dropped the managed-instance context. 
Since Topic writes now require `instanceId`, rebuilding a persisted Topic 
without a Broker route failed with HTTP 400; the follow-up route refresh was 
also unscoped.
   
   ## Brief change log
   
   - derive the rebuild target from the Topic record's `instanceId`, with the 
current selection as a legacy fallback
   - forward that same instance ID to both `createTopic` and `getTopicRoutes`
   - add a focused Topic page regression test covering the complete rebuild flow
   
   ## Verification
   
   - `NODE_OPTIONS=--no-experimental-webstorage npm test -- 
src/pages/instance/__tests__/TopicPage.test.tsx` — 9 tests passed
   - targeted ESLint passed with the base branch's existing 
`react-hooks/set-state-in-effect` finding disabled
   - `npm run build` passed
   - `git diff --check` passed
   


-- 
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]

Reply via email to