lizhimins commented on PR #4357: URL: https://github.com/apache/rocketmq-dashboard/pull/4357#issuecomment-5694517790
Validating the properties during the L2 preview rather than only at execution is the right place for the check. The fix itself is sound and comes with a test, so this is not a quality call. We have stopped merging one-defect-per-PR patches out of this queue: the review and merge overhead per change is high, and several of them touch files that other open PRs also touch. Please regroup your open Studio fixes into one PR per functional domain (for example instance/cluster persistence, alert rendering, AI tool input validation) and resubmit against the current `rocketmq-studio` baseline — we will review the consolidated PRs. When you regroup, note that other authors' patches on the same files are also being closed, so please pick one approach per file rather than resubmitting the overlap. -- 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]
