Skip to content

Conversation

@mtsgrd
Copy link
Contributor

@mtsgrd mtsgrd commented Jan 19, 2026

Remove the obsolete "Allow force pushing" toggle and its handler from the GitForm UI. The project property ok_with_force_push is no longer exposed in the form; only force_push_protection remains. This cleans up unused UI and event handling related to force-push toggling.

Copilot AI review requested due to automatic review settings January 19, 2026 13:27
@vercel
Copy link

vercel bot commented Jan 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

1 Skipped Deployment
Project Deployment Review Updated (UTC)
gitbutler-web Skipped Skipped Jan 19, 2026 1:50pm

Request Review

@mtsgrd
Copy link
Contributor Author

mtsgrd commented Jan 19, 2026

@krlvi just fyi, I noticed this setting is now unused by the backend.

Copy link
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR removes the obsolete "Allow force pushing" UI toggle from the GitForm component. The ok_with_force_push project property is no longer exposed in the settings form, as only force_push_protection is actively used. The PR cleanly removes the toggle UI component and its associated event handler function.

Changes:

  • Removed onForcePushClick handler function that updated ok_with_force_push
  • Removed CardGroup.Item containing the "Allow force pushing" toggle UI and its description
  • Simplified the GitForm to only show the "Force push protection" toggle

Remove the obsolete "Allow force pushing" toggle and its handler from
the GitForm UI. The project property ok_with_force_push is no longer
exposed in the form; only force_push_protection remains. This cleans up
unused UI and event handling related to force-push toggling.
@mtsgrd mtsgrd force-pushed the remove-ok-with-force-push-toggle branch from 8c852c9 to cfbe5a6 Compare January 19, 2026 13:50
@mtsgrd
Copy link
Contributor Author

mtsgrd commented Jan 19, 2026

image

@mtsgrd mtsgrd merged commit 9e36a8e into master Jan 19, 2026
23 checks passed
@mtsgrd mtsgrd deleted the remove-ok-with-force-push-toggle branch January 19, 2026 14:20
@krlvi
Copy link
Member

krlvi commented Jan 19, 2026

perfect

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants