Skip to content
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

Bunchafixes #516

Merged
merged 2 commits into from
Jan 2, 2025
Merged

Bunchafixes #516

merged 2 commits into from
Jan 2, 2025

Conversation

LTS-FFXIV
Copy link
Member

This pull request introduces several changes to the RotationSolver project, focusing on configuration updates, new feature additions, and code improvements. The most important changes include the addition of a new configuration option, the refactoring of AutoStatusOrder handling, and updates to the UI and rotation logic.

Configuration updates:

Refactoring AutoStatusOrder handling:

  • Removed AutoStatusOrder from Configs.cs and added it to OtherConfiguration.cs as a HashSet<uint>. (RotationSolver.Basic/Configuration/Configs.cs, [1]; RotationSolver.Basic/Configuration/OtherConfiguration.cs, [2]
  • Updated methods to initialize, save, and reset AutoStatusOrder in OtherConfiguration.cs. (RotationSolver.Basic/Configuration/OtherConfiguration.cs, [1] [2] [3]

UI updates:

  • Modified DrawAutoStatusOrderConfig to use the new AutoStatusOrder from OtherConfiguration and updated the reset functionality. (RotationSolver/UI/RotationConfigWindow.cs, [1] [2] [3] [4]
  • Updated WelcomeWindow to check the new ChangelogPopup configuration before displaying the changelog. (RotationSolver/UI/WelcomeWindow.cs, [1] [2]

Rotation logic updates:

Updated .gitignore to include Resources/AutoStatusOrder.json.
Moved AutoStatusOrder to OtherConfiguration as HashSet<uint>.
Added new config options _autoOffWhenDeadPvP and _changelogPopup.
Modified OtherConfiguration to handle AutoStatusOrder.
Added methods ResetAutoStatusOrder and SaveAutoStatusOrder.
Added partial method ModifyShirkPvE in CustomRotation_Actions.cs.
Updated RSCommands_Actions.cs for _autoOffWhenDeadPvP check.
Refactored RotationConfigWindow to use OtherConfiguration.
Updated WelcomeWindow to show changelog popup conditionally.
Adjusted StateUpdater to handle AutoStatusOrder as uint.
Updated assembly version in WelcomeWindow.cs to 7.1.5.24.
Added using directive for RotationSolver.Basic.Configuration.
@LTS-FFXIV LTS-FFXIV merged commit 7e0229d into main Jan 2, 2025
@LTS-FFXIV LTS-FFXIV deleted the bunchafixes branch January 3, 2025 16:32
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