search

Luke100000 / minecraft-comes-alive

Replaces Minecraft's villagers with humans, who can be interacted with.
GNU General Public License v3.0
70 stars 49 forks source link

Unable to Trade with Villagers #899

Closed OfficialDio closed 2 months ago

OfficialDio commented 2 months ago

Describe the bug When approaching a villager and talking to them, they give all the interactable options apart from trading.

To Reproduce Steps to reproduce the behavior:

  1. Go to a Villager
  2. Click on interact
  3. No Trade option apart from Gift, Pick up, locations, follow me, stay here, ride horse, armor
  4. See error

Screenshots image

Version 7.5.14, Minecraft 1.20.1, Curseforge

Latest.log

latest.log

Luke100000 commented 2 months ago

That's a guard, guards don't trade. Do you have a suggestion on how to remove this confusion?

OfficialDio commented 2 months ago

Yeah I just noticed when trading I've been trying to trade with Jobless Villagers... My apologies. A way to possibly remove the confusion though is to still add the trade option but gray it out like this

image

Adding a text pop up if hovering over it saying you cant trade with a Guard/Jobless villager. But showing unable to trade with xyzType of villager would most likely resolve potential confusion! Thank you for your speedy reply!