Skip to content

Remove sync plans for count mismatch - #1256

Merged
Joe-Ortiz merged 2 commits into
mainfrom
RemoveSyncPlansForCountMismatch
Jun 18, 2026
Merged

Remove sync plans for count mismatch#1256
Joe-Ortiz merged 2 commits into
mainfrom
RemoveSyncPlansForCountMismatch

Conversation

@Joe-Ortiz

Copy link
Copy Markdown
Member

Copilot Summary

This pull request updates the application version to 3.1.10 and removes the "Sync All Plans" functionality from the FrmDoublesDiscrepancies form. The changes simplify the UI and related logic by eliminating the button, its event handler, and associated code paths for syncing plans.

Version Update:

  • Updated application version to 3.1.10 in both pubxml files and updated displayed version text in frmMain.cs and FrmMainMenu.cs. [1] [2] [3] [4]

Removal of "Sync All Plans" Feature:

  • Removed the btnSyncAllPlans button and its initialization from FrmDoublesDiscrepancies.cs. [1] [2] [3]
  • Eliminated the BtnSyncAllPlans_Click event handler and the SyncPlan helper method. [1] [2]
  • Removed logic related to syncing plans in discrepancy rows and the bulk button updater. The "Sync Plan" action and enabling logic are no longer present. [1] [2] [3]

joeprogrammer88 added 2 commits June 18, 2026 09:17
Removed the "Sync All Plans" button, its event handler, and the SyncPlan method from FrmDoublesDiscrepancies. Disabled the "Sync Plan" fix for count mismatches and cleaned up related UI logic. Now only reciprocal discrepancies can be fixed from the interface.
Updated the application version from 3.1.9 to 3.1.10 in ClickOnceProfile.pubxml and updated the window titles in FrmMain and FrmMainMenu to reflect the new version, ensuring consistency between published and displayed versions.
@Joe-Ortiz
Joe-Ortiz merged commit e01c447 into main Jun 18, 2026
1 check passed
@Joe-Ortiz
Joe-Ortiz deleted the RemoveSyncPlansForCountMismatch branch June 18, 2026 17:21
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant