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

Add 100% shock avoidance when using tempest shield #6582

Closed
1 task done
moxaj opened this issue Sep 2, 2023 · 4 comments
Closed
1 task done

Add 100% shock avoidance when using tempest shield #6582

moxaj opened this issue Sep 2, 2023 · 4 comments
Labels
enhancement New feature, calculation, or mod

Comments

@moxaj
Copy link

moxaj commented Sep 2, 2023

Check for duplicates

  • I've checked for duplicate issues by using the search function of the issue tracker

Is your feature request related to a problem?

Currently, Arctic Armour gives 100% freeze avoidance:
image

Tempest Shield has the exact same wording (Buff grants immunity to X), however its avoidance does not show up in the calcs tab.

Describe the solution you'd like

When using Tempest Shield, pob should show your build having 100% shock avoidance.

Describe alternatives you've considered

No response

Additional context

pob link: https://pobb.in/j2pjs9pgL4MN

@moxaj moxaj added the enhancement New feature, calculation, or mod label Sep 2, 2023
@Lilylicious
Copy link
Contributor

While the reported issue is incorrect, it does highlight a bug. Arctic Armour should not give freeze avoid, but rather freeze immunity. A while back a clarification was made to all ailments that immunity is not the same as 100% avoidance.

@moxaj
Copy link
Author

moxaj commented Sep 2, 2023

I see, but still, I'd expect those buffs to show up in the calcs tab, in whatever form.

@LocalIdentity
Copy link
Contributor

The bug issue is correct and Artic Armour does need to be changed to show Immunty instead of Avoid. I'll put up a fix for both issues in a min

@Paliak
Copy link
Contributor

Paliak commented Sep 15, 2023

Fixed by #6584

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature, calculation, or mod
Projects
None yet
Development

No branches or pull requests

4 participants