Skip to content

Add psion battlerage sorting#78

Merged
keneanung merged 2 commits intoAchaeaBashingScript:masterfrom
tsartin:add-psion-battlerage-sorting
Aug 30, 2021
Merged

Add psion battlerage sorting#78
keneanung merged 2 commits intoAchaeaBashingScript:masterfrom
tsartin:add-psion-battlerage-sorting

Conversation

@tsartin
Copy link
Contributor

@tsartin tsartin commented Aug 16, 2021

No description provided.

@tsartin tsartin changed the title Add files via upload Add psion battlerage sorting Aug 16, 2021
Copy link
Member

@keneanung keneanung left a comment

Choose a reason for hiding this comment

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

Hey there. Thank you for your contribution! I have a question to ask before I merge this.

if class ~= "Psion" or #battlerageSkills ~= 6 then return end
battlerageSkills[2], battlerageSkills[3], battlerageSkills[4] = battlerageSkills["regrowth"], battlerageSkills["pulverise"], battlerageSkills["devastate"]

battlerageSkills["terror"].affliction = "fear"
Copy link
Member

Choose a reason for hiding this comment

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

Is this the only battlerage ability that is missing the affliction? Does the "combo" ability have the used afflictions set?

@keneanung keneanung merged commit 4ecd10e into AchaeaBashingScript:master Aug 30, 2021
@tsartin
Copy link
Contributor Author

tsartin commented Aug 31, 2021

Apologies for the late reply. been a little out of pocket.

I don't have psion myself. I'm thinking that terror was the only one that was missing an affliction and the other person didn't think the combo ability used an affliction. This is the information they shared though.

"Barbedblade does DoT damage."
"Devastate does damage."
"Regrowth calls forth binding vines to grasp your foe, preventing them from being able to heal. no aff or requirements lists."
"Pulverise breaks shield"
"Whirlwind says causes massive damage. no aff or anything required."
"Then terror gives fear"

@keneanung
Copy link
Member

All right. I was thinking I'd merge this as it's an improvement and then fix things on the go.

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.

2 participants