fix: Auto-Approval Enabled checkbox now enables Save button #10754
+11
−2
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This PR attempts to address Issue #10745. Feedback and guidance are welcome.
Summary
This PR fixes the inconsistent behavior where toggling the "Auto-Approval Enabled" checkbox in the Auto-Approve settings tab did not enable the Save button, making it behave differently from other checkboxes in the settings.
Changes
AutoApproveSettings.tsx:
autoApprovalEnabledprop from cached statesetCachedStateField("autoApprovalEnabled", newValue)to enable Save button activationsetAutoApprovalEnabled(newValue)call for immediate timer cancellation (hybrid approach)SettingsView.tsx:
autoApprovalEnabledto the destructuredcachedStateautoApprovalEnabledprop toAutoApproveSettingscomponentautoApprovalEnabledtohandleSubmitfor persistence when Save is clickedHybrid Approach
This implementation uses a hybrid approach that:
The setting is persisted when the user clicks the Save button.
Testing
Closes #10745
Important
Fixes issue with 'Auto-Approval Enabled' checkbox not enabling Save button by updating state management in
AutoApproveSettings.tsxandSettingsView.tsx.Auto-Approval Enabledcheckbox did not enable Save button inAutoApproveSettings.tsx.autoApprovalEnabledtocachedStateinSettingsView.tsx.handleSubmitinSettingsView.tsxto persistautoApprovalEnabled.SettingsViewandFollowUpSuggesttests pass.This description was created by
for 18f86e7. You can customize this summary. It will automatically update as commits are pushed.