-
Notifications
You must be signed in to change notification settings - Fork 303
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
Ulduar: Convert Flame Leviathan to BossAI & Spell Lists #429
base: master
Are you sure you want to change the base?
Conversation
OK I consider this PR as functionally complete.
I don't have the patience to fix these issues right now and I'd appreciate it a lot if someone else would consider helping me 👍 |
@@ -174,7 +174,7 @@ void TemporarySpawn::Update(const uint32 diff) | |||
} | |||
if (!GetCharmerGuid()) | |||
{ | |||
if (IsExpired()) | |||
if (IsExpired() && !IsInCombat()) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
cdc98da
to
e3be7e4
Compare
a6bf96b
to
303d230
Compare
For easier review I created a new PR on my fork which has the Meson commits stripped and the Leviathan commits squashed: https://github.com/insunaa/mangos-wotlk/pull/1/files Please someone take the 30 minutes it takes to review. Thank you! |
- Fix passenger sometimes visually not mounting - Fix wrong seat coordinate if the player is the vehicle - Deduplicate some code
Any news on this one @insunaa, I mean since my last test. Cause this one would fix a lot of problems in game when merged |
@al3xc1985 I'm catching up on work due to illness right now, so little time for this, hopefully soon. |
Hope to get well! God bless! |
🍰 Pullrequest
This PR converts Flame Leviathan to BossAI and adds Spell Lists
Todo / Checklist
Random notes: