Aias00 opened a new issue, #913:
URL: https://github.com/apache/rocketmq-dashboard/issues/913

   ### Description
   
   In mock mode, `updateAlertRule()` returns the submitted alert rule even when 
no existing rule has the requested `id`.
   
   ### Location
   
   `web/src/services/opsService.ts`
   
   ### Impact
   
   The Alert Management page can show a successful save for a stale or deleted 
alert rule ID even though no rule was updated. This hides stale state and makes 
the mock service contract diverge from real API behavior.
   
   ### Suggested fix
   
   Throw an `Alert rule not found` error when `updateAlertRule()` cannot find 
the target rule. Add a regression test that attempts to update an unknown rule 
ID and verifies that the alert rule list is unchanged.
   
   ### Related existing work
   
   #757 fixed the same missing-ID pattern for ACL rules, but it does not cover 
alert rules in `opsService`.
   
   ### Scope
   
   RocketMQ Studio Track 1 / Ops alert management.


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