Aias00 opened a new pull request, #4180:
URL: https://github.com/apache/rocketmq-dashboard/pull/4180

   ## Summary
   
   - add read-only alert-rule import previews for Cluster and Business alert 
domains
   - classify rows as `NEW`, `DUPLICATE`, or `INVALID` with row-level 
diagnostics
   - apply imports with deterministic `FAIL`, `SKIP`, or `REPLACE` conflict 
handling
   - keep the legacy `/import` endpoint compatible with `FAIL` behavior
   - add a two-step localized UI that guards against stale preview responses
   
   ## Safety
   
   - validate all rows before writing
   - rebuild the import plan inside the apply transaction
   - never delete rules during import
   - verify rollback with a real Spring/H2 transaction test that forces the 
second write to fail
   
   ## Verification
   
   - `npm test -- --run src/api/ops.test.ts src/services/opsService.test.ts 
src/pages/ops/__tests__/AlertsPage.test.tsx` (67 passed)
   - `npm run build`
   - changed-scope ESLint (0 errors; existing warnings only)
   - `mvn 
-Dtest=AlertRuleTransferServiceTest,AlertRuleTransferTransactionTest,AlertRuleControllerTest,ClusterAlertRuleControllerTest
 test` with JDK 21 (41 passed, checkstyle 0 violations)
   - `git diff --check`
   
   Full repository suites were also exercised. The frontend suite had one 
unrelated `ConsumerPage` concurrency test fail once and pass when rerun alone. 
The backend suite reported three existing failures in `AuthCorsIntegrationTest` 
and `AliyunInstanceProviderTest`; none of those files are changed here.
   
   Fixes #4178
   


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