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

PLD Beta update and AOE Heal correction #520

Merged
merged 1 commit into from
Jan 3, 2025
Merged

PLD Beta update and AOE Heal correction #520

merged 1 commit into from
Jan 3, 2025

Conversation

LTS-FFXIV
Copy link
Member

This pull request includes several updates to the PLD_Beta class in BasicRotations/Tank/PLD_Beta.cs, changes to the AutoStatus enum in RotationSolver.Basic/Data/AutoStatus.cs, and a modification to the ModifyCircleOfScornPvE method in RotationSolver.Basic/Rotations/Basic/PaladinRotation.cs. The most important changes are summarized below:

Updates to PLD_Beta class:

  • Added a new configuration option UseHolyWhenAway to allow the use of Holy Spirit when out of melee range.
  • Modified the AttackAbility method to skip the AoE check when using BladeOfHonorPvE.
  • Updated the GeneralGCD method to respect the new UseHolyWhenAway configuration option when deciding to use Holy Spirit.

Changes to AutoStatus enum:

  • Corrected the descriptions and values for healing abilities and spells, swapping single and area healing statuses.

Modification to ModifyCircleOfScornPvE method:

  • Added a TimeToKill property with a default value of 0 in the ActionConfig for CircleOfScornPvE.

- Added `UseHolyWhenAway` to `PLD_Beta` for using Holy Spirit when out of melee range.
- Modified `AttackAbility` in `PLD_Beta` to skip AoE check for `BladeOfHonorPvE`.
- Updated `HolySpiritPvE` condition in `PLD_Beta` to check `UseHolyWhenAway`.
- Corrected `AutoStatus` enum descriptions and names for healing abilities.
- Added `TimeToKill` property to `ActionConfig` in `PaladinRotation` with a default value of 0.
@LTS-FFXIV LTS-FFXIV merged commit 126872b into main Jan 3, 2025
@LTS-FFXIV LTS-FFXIV deleted the PLDandAOE branch January 4, 2025 11:38
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