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

Fixes #14461: Avatar stats don't update after another user uses a buffing abililty #14920

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

ArbiterBibendi
Copy link

Fixes #14461
Party members' state wasn't always being updated when casting spells.
If the spell is is a bulk spell (bulk: true), the state would not update when the spell is cast. The state was being updated during casting in website\common\script\content\spells.js until bulk spells were introduced in #14544.

Changes

Updates the partyMembers state using the updated data returned by the cast API call
in the castEnd() method in website\client\src\mixins\spells.js rather than relying on each spell's cast method in the common scripts to update it.


UUID: 00cfc368-4bf7-4d50-8100-6e3f21707e10

@ArbiterBibendi
Copy link
Author

ArbiterBibendi commented Oct 6, 2023

I'm a first time contributor so the workflow for tests hasn't run automatically. Please may an admin start it? 👉👈

@ArbiterBibendi
Copy link
Author

Hello, I'm just leaving a comment again because it's been a while.

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.

Avatar stats don't update after another user uses a buffing abililty
1 participant