-
-
Notifications
You must be signed in to change notification settings - Fork 2
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]: Merge Pets non-functional in Patch 10.2 #10
Comments
Confirm. Merge pets option works incorrectly. |
Hello @wordup834 . Thanks for reporting this. It should be |
Unfortunately not, issue still present using 3.4.x - issue possibly related to requiring server name in owner name now following cross faction guilds? |
I took a look at the changes and noticed that the game is always adding the realm's name even if players are from the same realm. If you could test this file out it would be great! Simply drop it inside |
This fixes the issue yes, looks like it's merging pets again, Thanks for the fix! |
Seems to be broken once again for me - i have tried both the v1.8.85 + init file and the v1.8 .86 + init file. |
Thanks for reporting this @ArenfeIdt .. I could test the addon after ages of updates and I could confirm it. Try to update the addon from the mega folder, I believe I fixed it. |
Yes, it works again now - however, the Version in the Mega Folder on its own does not work for me. Only after adding the Init.lua File you posted earlier in this Issue Report does it work. Just tested it with the Priest Shadowfiend, BM Hunter Pets and the Voidwalker Warlock Pet. All of them merge properly now. Probably better if you add the Init.lua File directly to the new Version ZIP. |
There isn't a big difference between both files. However, the file I posted had an issue, it was using the player's realm name (yours) even for players from other realms, so I just edited that part to use the realm name of each player. |
Maybe something is different because i use the Setting to not display Realm Names. |
It shouldn't affect anything, that option simply strips off the realm name, for display purposes, not data. |
Hmm, i just know that if i use the Init file that comes with the entire Skada Folder it doenst work for me - only after using the extra init file you posted does it work for me. |
Are you using the latest version of the addon?
Addon Version
Version 1.8.85 - Date: 2023-11-09 @ 07:30 GMT
Did you try having only this addon enabled?
Current Behavior
When pets are summoned, the owner/player is no longer the primary damage source instead of being merged together. They are instead bumped to number 2, and the pet that triggers the error also does not appear on the main window either.
Expected Behavior
Pets to be merged as one combined result.
Lua Error
N/A
Reproduction Steps
Have merge pets enabled
Enter Combat
Summon any temporary guardian
Last Good Version
10/30/2023
Screenshots
The text was updated successfully, but these errors were encountered: