Fix when removing all the values in the expression editor causes the …#963
Conversation
…value to be reset
|
Important Review skippedAuto reviews are disabled on base/target branches other than the default branch. Please check the settings in the CodeRabbit UI or the You can disable this status message by setting the ✨ Finishing touches🧪 Generate unit tests (beta)
Tip 📝 Customizable high-level summaries are now available in beta!You can now customize how CodeRabbit generates the high-level summary in your pull requests — including its content, structure, tone, and formatting.
Example instruction:
Note: This feature is currently in beta for Pro-tier users, and pricing will be announced later. Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out. Comment |
Purpose
Fix an issue in the Expression Editor where removing all values caused the field to be unexpectedly reset.
Resolves: wso2/product-ballerina-integrator#1953
Goals
Ensure that clearing all values in the Expression Editor does not trigger a reset of the value. The editor should correctly reflect an empty state when all inputs are removed.
Approach
Removed the unnecessary value comparison inside the
onChangehandler, which was causing the value to reset when clearing all inputs.Now, the Expression Editor correctly updates its state without reverting to a previous value.