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 AoE Avoidance for Hot Coals in MC #6

Merged
merged 1 commit into from
May 20, 2024

Conversation

Day36512
Copy link

@Day36512 Day36512 commented May 13, 2024

Majordomo Hot Coals damage avoidance.

Majordomo Hot Coals damage avoidance.
@trickerer
Copy link
Owner

This is ok but why? Bots don't take damage from hot coals since it's a non-hunter trap type gameobject which does environmental damage and only targets players.

@Day36512
Copy link
Author

Because tanks will simply sit in the coals if they end up there which is painful for melee players. I suppose a simple sendto command would work but this just avoids the issue altogether.

@kissingers
Copy link

sorry I have a crash but noway submit ,just add here
https://pastebin.com/9bWp6TMs

it seems crash from /src/server/game/AI/NpcBots/bot_ai.cpp:18636
now just at line 18637 add this but dont know is it right?
if (!mineWP)
continue;

@trickerer
Copy link
Owner

sorry I have a crash but noway submit ,just add here https://pastebin.com/9bWp6TMs

it seems crash from /src/server/game/AI/NpcBots/bot_ai.cpp:18636 now just at line 18637 add this but dont know is it right? if (!mineWP) continue;

Issue tracker is here: https://github.com/trickerer/Trinity-Bots/issues. Your issue I think is trickerer/Trinity-Bots#616

@trickerer trickerer merged commit 0bd1498 into trickerer:npcbots_3.3.5 May 20, 2024
12 checks passed
ikkj pushed a commit to ikkj/AzerothCore-wotlk-with-NPCBots that referenced this pull request Jun 10, 2024
Majordomo Hot Coals damage avoidance.
junimslage10 pushed a commit to junimslage10/AzerothCore-wotlk-with-NPCBots that referenced this pull request Jun 27, 2024
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.

3 participants