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

Something is off about Load Conditions with Spell Known #4638

Closed
2 tasks done
ljosberinn opened this issue Oct 9, 2023 · 1 comment
Closed
2 tasks done

Something is off about Load Conditions with Spell Known #4638

ljosberinn opened this issue Oct 9, 2023 · 1 comment
Labels
⏱ Awaiting Response This ticket hasn't been triaged yet. 🐛 Bug This is a problem with WeakAuras.

Comments

@ljosberinn
Copy link

ljosberinn commented Oct 9, 2023

Is there an existing issue for this?

  • I have searched the existing open and closed issues.

Description

This WA should show Upheaval while talented into Font of Magic: https://wago.io/WGqPBXKqL
Since the latest release, it no longer loads:
image

This WA should show Upheaval while NOT talented into Font of Magic: https://wago.io/zjJRJ2eFU
Since the latest release, it loads when it shouldnt:
image

I'm not sure whether it always behaved like this in terms of IsSpellKnown: confirmed it behaves identical on PTR so likely no change here
image

Meanwhile on PTR, WA 5.7.0 works fine:
image

The actual cast spell id in combat log is also accurate: https://www.warcraftlogs.com/reports/MRczQjFaLC1ygYZb#fight=2&type=casts&source=18&ability=408092

WeakAuras Version

5.8.0

World of Warcraft Flavor

Retail (Default)

World of Warcraft Region

EU

Tested with only WeakAuras

  • Yes

Lua Error

No response

Reproduction Steps

all in the description, im sorry

Last Good Version

5.7.0

Screenshots

No response

Export String

No response

@ljosberinn ljosberinn added the 🐛 Bug This is a problem with WeakAuras. label Oct 9, 2023
@github-actions github-actions bot added the ⏱ Awaiting Response This ticket hasn't been triaged yet. label Oct 9, 2023
@mrbuds
Copy link
Contributor

mrbuds commented Oct 9, 2023

This is solved after reverting 68b6d98

mrbuds added a commit to mrbuds/WeakAuras2 that referenced this issue Oct 9, 2023
mrbuds added a commit to mrbuds/WeakAuras2 that referenced this issue Oct 9, 2023
mrbuds added a commit to mrbuds/WeakAuras2 that referenced this issue Oct 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
⏱ Awaiting Response This ticket hasn't been triaged yet. 🐛 Bug This is a problem with WeakAuras.
Projects
None yet
Development

No branches or pull requests

2 participants