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

[PVP] Changes and fixes wave 2 #113

Closed
wants to merge 12 commits into from
Closed

Conversation

edewen
Copy link
Contributor

@edewen edewen commented Nov 25, 2024

DNC
ported the dancer partner remind over from pve without the autorotation stuff. added EnAvant option to boost 1 2 combo damage with slider to save charges per user preference.

BLM
Fixed the blizzard side trying to use foul. Not longer a pvp spell. Replaced with xeno as it intends
Fixed missing slider for wreath of ice health threshold.
Added wreath of fire execute mode option with slider. While it goes through guard, it is a lot of dps lost when it can have 50% uptime.

SAM

MNK
Fixed logic of fire resonance to always use it with phantom rush (largest hit of 12k) unless 2 charges are available and then throw one of a flints or winds reply (2nd largest hits of 8k but aoe)

Dncer, ported the dancer partner remind over from pve without the autorotation stuff. added ev avant option to boost 1 2 combo damage with slider to save charges per user preference.
@edewen edewen marked this pull request as draft November 25, 2024 00:27
@zbee zbee added the Fixes This will fix a current issue label Nov 25, 2024
Code had foul instead of xeno. Foul is gone so fixed.

Added the missing slider for wreath of ice health threshold.

Added wreath of fire execute mode option for targets beneath 20k health.
Fixed Logic so Fire resonance is always spent on phantom rush for the 18k hit. Otherwise it will be spent on wind's or flints reply when a second charge is available.
@edewen edewen marked this pull request as ready for review November 25, 2024 03:30
zbee and others added 3 commits November 25, 2024 06:20
# Conflicts:
#	WrathCombo/Combos/PvE/BLM/BLM_Config.cs
#	WrathCombo/Combos/PvP/DNCPVP.cs
Fix ice that broke with zbees merge. Turned fire execute into a slider
Copy link
Contributor

@Kaeris-Tempest Kaeris-Tempest left a comment

Choose a reason for hiding this comment

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

I don't play MNK so I can't help much on that one.

BLM, besides what I pointed out, doesn't have charges on Aetherial Manipulation anymore so that can also be adjusted.

WrathCombo/Combos/PvP/BLMPVP.cs Show resolved Hide resolved
WrathCombo/Combos/PvP/DNCPVP.cs Show resolved Hide resolved
Added the umbral ice check. Worked without but still needed cleanup.

changed aetherial manip from charges to actionready
removed actionready. it wasnt needed
Mimiced the st changes on aoe button as well
@edewen edewen closed this Nov 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Fixes This will fix a current issue
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants