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

Update ECommons submodule and refactor Configs class #424

Merged
merged 1 commit into from
Oct 5, 2024

Conversation

LTS-FFXIV
Copy link
Member

Updated ECommons submodule to commit b39bbc68ff7ee632d3121388e6f4e4aad6753dcd. Removed several configuration options from the Configs class:

  • _changeTargetForFate
  • _useTasksForOverlay
  • _onlyHotOnTanks
  • _targetFatePriority Replaced ConditionBool attribute with JobConfig for:
  • _interruptibleMoreCheck
  • _raisePlayerByCasting
  • _raisePlayerBySwift
  • _raiseBrinkOfDeath Changed PvEFilter for _raisePlayerBySwift to JobFilterType.Raise. Removed condition checks in DataCenter.cs and ObjectHelper.cs. Removed VPR job condition checks in CustomRotation_Ability.cs for JobRole.Melee. Added feature in CheckBoxSearch.cs to draw job icon if IsJob is true.

Updated ECommons submodule to commit b39bbc68ff7ee632d3121388e6f4e4aad6753dcd.
Removed several configuration options from the Configs class:
  - _changeTargetForFate
  - _useTasksForOverlay
  - _onlyHotOnTanks
  - _targetFatePriority
Replaced ConditionBool attribute with JobConfig for:
  - _interruptibleMoreCheck
  - _raisePlayerByCasting
  - _raisePlayerBySwift
  - _raiseBrinkOfDeath
Changed PvEFilter for _raisePlayerBySwift to JobFilterType.Raise.
Removed condition checks in DataCenter.cs and ObjectHelper.cs.
Removed VPR job condition checks in CustomRotation_Ability.cs for JobRole.Melee.
Added feature in CheckBoxSearch.cs to draw job icon if IsJob is true.
@LTS-FFXIV LTS-FFXIV merged commit 6d24a19 into main Oct 5, 2024
@LTS-FFXIV LTS-FFXIV deleted the jobconfigandtargets branch October 19, 2024 15:06
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