fix invalid form status update in new variable creation of addConnect…#966
Conversation
|
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 Create Connection flow where adding a new variable on the fly would break the form.
Resolves: wso2/product-ballerina-integrator#1869
Goals
Ensure that users can add a new variable without breaking the Create Connection form. The new variable form should save independently, without interfering with the main Add Connection form.
Approach
Removed the unnecessary
setSavingFormStatusstate update in the Create Connection form. This state was being set while creating a new variable, but it is not needed because we are saving the variable form that opens on top of the Add Connection form, not the original form itself.This prevents the Add Connection form from incorrectly entering a saving state when a new variable is added.