-
Notifications
You must be signed in to change notification settings - Fork 4.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
fix: auto saving config #3395
fix: auto saving config #3395
Conversation
Pull Request Validation ReportThis comment is automatically generated by Conventional PR Whitelist Report
Result Pull request does not satisfy any enabled whitelist criteria. Pull request will be validated. Validation Report
Result Pull request satisfies all enabled pull request rules. Last Modified at 16 Aug 24 19:28 UTC |
This pull request is automatically being deployed by Amplify Hosting (learn more). |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM
This pull request adds an auto-saving feature to the application. It includes several commits that introduce the necessary changes. The
useSaveConfig
hook is added to handle the configuration settings related to auto-saving. TheuseFlowsManagerStore
is used to manage theautoSaving
state. TheMenuBar
component is updated to disable the save button when auto-saving is enabled. TheFlowSettingsModal
component is updated to save the flow automatically when auto-saving is enabled. TheFlowPage
component is updated to enable auto-saving based on the configuration settings. TheFlowsManagerStoreType
is updated to include theautoSaving
property and thesetAutoSaving
function. Thevite.config.js
file is updated to remove theLANGFLOW_AUTO_SAVING
environment variable.