Skip to content

Change watcher's eye dodge mods to suppress mods#5562

Merged
Wires77 merged 1 commit intoPathOfBuildingCommunity:devfrom
deathbeam:fix-dodge
Jan 6, 2023
Merged

Change watcher's eye dodge mods to suppress mods#5562
Wires77 merged 1 commit intoPathOfBuildingCommunity:devfrom
deathbeam:fix-dodge

Conversation

@deathbeam
Copy link
Contributor

  • Haste one is legacy but in standard its suppress spell damageA
  • Grace is now suppress spell damage not dodge attacks

Signed-off-by: Tomas Slusny slusnucky@gmail.com

Before screenshot:

image

After screenshot:

image

@Wires77
Copy link
Member

Wires77 commented Jan 6, 2023

This is actually generated automatically now. https://github.com/PathOfBuildingCommunity/PathOfBuilding/blob/dev/src/Data/Uniques/Special/Generated.lua#L565

I'm not actually sure how you got the dodge mods to appear since the generated version of the jewel should take precedence...were you on a really old version by chance?

Regardless, the proper fix would be to simply remove Watcher's Eye from jewel.lua.

@deathbeam
Copy link
Contributor Author

Oh i see. And well im on latst dev branch so not sure, they are just there.

This fixes these mods:
- Haste one is legacy but in standard its suppress spell damageA
- Grace is now suppress spell damage not dodge

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
@deathbeam
Copy link
Contributor Author

Alright removed watchers eye from the file instead and it looks like that works too

@Wires77
Copy link
Member

Wires77 commented Jan 6, 2023

What OS are you on? I assume it behaves correctly with other generated uniques and if you remove watchers eye from jewel.lua?

@deathbeam
Copy link
Contributor Author

deathbeam commented Jan 6, 2023

linux (arch), with pobfrontend. And yea after i removed watchers eye from jewel.lua it behaves correctly, i am not sure about other generated uniques. I checked on windows without pobfrontend and there its behaving fine. I guess its just some scuffed load order on pobfrontend then. Still removing it from the file probs do not hurts anyone and at least fixes it on linux i guess?

@Wires77
Copy link
Member

Wires77 commented Jan 6, 2023

Yeah, this looks good to me, I may check on the underlying issue at some point, but in reality this shouldn't really happen anyway

@Wires77 Wires77 merged commit 92f13a5 into PathOfBuildingCommunity:dev Jan 6, 2023
@deathbeam deathbeam deleted the fix-dodge branch January 6, 2023 22:38
Dullson pushed a commit to Dullson/PathOfBuilding that referenced this pull request Dec 6, 2023
…Community#5562)

This fixes these mods:
- Haste one is legacy but in standard its suppress spell damageA
- Grace is now suppress spell damage not dodge

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>

Signed-off-by: Tomas Slusny <slusnucky@gmail.com>
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