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

[Bug Fix] Fix Swarm Pet Flurry/Rampages Messages #2444

Merged
merged 2 commits into from
Sep 25, 2022
Merged

[Bug Fix] Fix Swarm Pet Flurry/Rampages Messages #2444

merged 2 commits into from
Sep 25, 2022

Conversation

Aeadoin
Copy link
Contributor

@Aeadoin Aeadoin commented Sep 23, 2022

Flurry & Rampage messages will now filter to proper chat window (Pet Flurry/Rampage filter) instead of using NPC Flurry/Rampage chat window filtering.

zone/mob_ai.cpp Outdated
target->GetCleanName());
}

if (target) {
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

These should be tabs, not spaces.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Should hopefully be fixed now

@Kinglykrab Kinglykrab changed the title Fix Swarm Pet Flurry/Rampages Messages [Bug Fix] Fix Swarm Pet Flurry/Rampages Messages Sep 23, 2022
@Kinglykrab
Copy link
Contributor

👍

@mackal
Copy link
Member

mackal commented Sep 23, 2022

I assume this is correct, but I could still see live EQ being dumb and just labeling them as NPCs for this case :P

@Kinglykrab Kinglykrab merged commit 2b203c0 into EQEmu:master Sep 25, 2022
@Aeadoin Aeadoin deleted the Aeadoin/FixSwarmFlurry branch September 25, 2022 17:07
catapultam-habeo pushed a commit to catapultam-habeo/pyrelight-server that referenced this pull request Mar 27, 2023
* Fix Swarm Pet Flurry/Rampages Messages

* Fixed formatting
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.

None yet

3 participants