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.

Sign up for GitHub

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

Jump to bottom

Crusade on Retribution Paladin #806

Closed
Hemario opened this issue Jan 9, 2021 · 0 comments
Closed

Crusade on Retribution Paladin #806

Hemario opened this issue Jan 9, 2021 · 0 comments
Assignees
@Sidoine
Labels
bug
Projects
Bugs

Comments

@Hemario
Copy link
Contributor

Hemario commented Jan 9, 2021 •
edited

The generate SpellInfo for Crusade is the following:
SpellInfo(crusade cd=20 charge_cd=120 duration=25 max_stacks=10 gcd=0 offgcd=1)

This is not correct, it should be:
SpellInfo(crusade cd=120 duration=25 max_stacks=10 gcd=0 offgcd=1)
As a result, Crusade is shown in the main icon (instead of cd icon)

The replaced_by is also missing in the file
SpellRequire(avenging_wrath replaced_by set=crusade enabled=(hastalent(crusade_talent)))
@Sidoine Sidoine self-assigned this Jan 11, 2021
@Sidoine Sidoine added the bug label Jan 11, 2021
@Sidoine Sidoine added this to To do in Bugs via automation Jan 11, 2021
Bugs automation moved this from To do to Done Jan 15, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Sidoine Sidoine

Labels
bug
Projects
Bugs
Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Sidoine @Hemario