search

TheCSMods / mc-better-stats

A Minecraft (Fabric) mod that improves the statistics screen and makes it more useful.
https://modrinth.com/mod/n6PXGAoM
GNU Lesser General Public License v3.0
49 stars 19 forks source link

[v3.3+fabric-1.20.2] Warden shouldn't be in the Monsters Hunted tab #88

Closed noawx closed 8 months ago

noawx commented 1 year ago

What happened?

I opened the Monsters Hunted tab to see what I needed to kill, and saw Warden there, despite Warden not being needed for the Monsters Hunted advancement.

Steps to reproduce

  1. Open the Statistics menu
  2. Go to the "Monsters Hunted" tab
  3. Issue occurs

Relevant log output or crash report

No response

Other installed mods

  1. AppleSkin
  2. Armor Trim Item Fix
  3. Auth Me
  4. Better Mount HUD
  5. Better Ping Display
  6. Chat Heads
  7. Controlify
  8. DeathLog
  9. Elytra Trims
  10. EMI
  11. Fabric API
  12. Fabric Language Kotlin
  13. Fluid Void Fading
  14. Forge Config API Port
  15. Jade
  16. Journeymap
  17. Just Enough Items
  18. Just Enough Professions
  19. LibjF
  20. Litematica
  21. MaLiLib
  22. Mod Menu
  23. NBT Autocomplete
  24. Not Enough Crashes
  25. oωo
  26. Resource Pack Options
  27. Server Pack Unlocker
  28. Shulker Box Tooltip
  29. Simple armor hud
  30. Status Effect Bars
  31. YetAnotherConfigLib
  32. Zoomify

Code of Conduct

TheCSDev commented 1 year ago

Hm interesting. I did not know warden was not a requirement. Thanks for letting me know. I'll deal with that after v3.4, as I just published v3.4 moments ago, and am working on getting it on CF and Modrinth.

TheCSDev commented 8 months ago

My goal was to tackle this issue, but then forgot about it. My initial idea was to include some sort of indicator that certain mobs aren't required, but ran into issues when finding out that knowing whether or not a mob is "required" is not so simple, as advancement creiteria can be controlled by mods, data-packs, and other server-side code. As such, I eventually decided to display everything and just rename the tabs to avoid confusion. Maybe someday I find a way to tackle the issue, and if I do, I'll likely add an indicator showing whether or not a mob is required for the advancement. Until then, I'll have no choice but to clise this issue as "not planned".
Thanks again, and have a great day!