RockteMQ-AI commented on issue #4280: URL: https://github.com/apache/rocketmq-dashboard/issues/4280#issuecomment-5655088231
**Issue Evaluation** Category: `type/bug` | Status: **Confirmed** The issue is valid. `LlmConfigService.saveConfig` rebuilds the full `GeneralSettingsVO` but omits `dingtalkWebhook`, `smsWebhook`, and `emailRecipients` when copying fields. This causes saving AI/LLM config to silently wipe notification channel settings. The fix should ensure all existing settings fields are preserved during the rebuild, not just the ones currently visible in the AI/LLM settings UI. A defensive approach would be to load the existing settings, apply only the changed fields, then save — rather than reconstructing from scratch. --- *Automated evaluation by github-manager* -- 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]
