search

Bloodmallet / bloodmallet_web_frontend

Frontend of bloodmallet.com
https://bloodmallet.com
1 stars 8 forks source link

Retribution Paladin Power Infusion support not detected #109

Closed cremor closed 1 year ago

cremor commented 1 year ago

Power Infusion support was added to the Retribution Paladin APL in https://github.com/simulationcraft/simc/commit/00f508e12e4a8d62b920b51d2c96c7a7274662ef but bloodmallet still says "No APL support." on https://bloodmallet.com/chart/power_infusion

edit: I just noticed that the table uses an older SimC version than the charts above. So that would explain it. But the point below is still valid.

Also, all specs link to a T29 profile in that table. The links should be upated to the T30 profiles. (And if bloodmallet actually still uses the T29 profiles for the charts than that should be updated too!)

Bloodmallet commented 1 year ago

Greetings, thank you for creating this issue.

I'm updating the Power Infusion table now.

The link to the T29 profiles was wrong. The charts use T30 profiles for alle data generation. This will be fixed with the next website update.

Bloodmallet commented 1 year ago

The above commit fixes the raised issue.