search

snowflame0 / AtlasLootClassic_Cata

AtlasLootClassic Cata
https://www.curseforge.com/wow/addons/atlaslootclassic
GNU General Public License v2.0
70 stars 34 forks source link

"Show learned info for companions" option doesn't work on cata pets #78

Closed Cabro closed 1 month ago

Cabro commented 1 month ago

AtlasLoot version 4.0.1-beta4

Describe the bug It fails to add the collected/not collected tooltips to cata pets.

To Reproduce Learn one of the cata pets and notice no "Collected" text on it

Expected behavior image

Screenshots image image

Additional context

Firsthunterz commented 1 month ago

https://github.com/snowflame0/AtlasLootClassic_Cata/pull/77/files this is the fix for that if you want to download the file and replace it.

snowflame0 commented 1 month ago

Sorry, been inactive for a couple weeks. I'll push out a new update soon. Should be ready to drop the beta tag.

Cabro commented 1 month ago

@Firsthunterz that fix is missing these ones:

   [34955] = {45890, 25706, 1}, --Scorched Stone
   [198665] = {384796, 194870, 1}, --Pebble

Also the fix is not working properly. I don't know why it shows "Collected" only on some of them, when i know them all: image

One last thing, you need to open the pet/mount journal in order for this table to update. Can you make it so you don't have to do that?