tju-yxq opened a new pull request, #4480:
URL: https://github.com/apache/rocketmq-dashboard/pull/4480

   ### Which Issue(s) This PR Fixes
   
   - Fixes #3556
   
   ### Brief Description
   
   Re-opens the previously reviewed change after the base branch moved from 
`rocketmq-studio` to `master`. Rebased onto current `master` (`987b748e`), all 
conflicts resolved, and focused tests re-run locally.
   
   cd server && mvn 
'-Dtest=AlertServiceTest,AlertRuleControllerTest,ClusterAlertRuleControllerTest,MybatisPlusAlertRepositoryTest'
 test → 124 tests passed; cd web && npm test -- AlertsPage.test.tsx 
alerts.test.ts --run → 40 tests passed; lint/build passed
   
   ### How Did You Test This Change?
   
   See the verification commands and results above; all were run on the rebased 
branch.
   
   ### Checklist
   
   - [x] One coherent change; unrelated modifications are not bundled in
   - [x] Commit subject follows Conventional Commits
   - [x] Tests added or updated for non-trivial changes, test methods named 
`...Test`
   - [x] New UI text has both Chinese and English entries under `web/src/i18n/`
   - [x] Architecture constraints stay green (`mvn test` runs the ArchUnit 
checks)
   - [x] New source files carry the ASF license header
   - [ ] Documentation touched where behaviour changed (README / `docs/` / 
in-app help)
   
   


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