PyvesB / advanced-achievements

:fireworks: Popular plugin that adds unique and challenging achievements to Minecraft servers.
https://www.spigotmc.org/resources/83466
GNU General Public License v3.0
199 stars 215 forks source link

Villager trade category update needed #949

Closed RJGamers closed 3 years ago

RJGamers commented 3 years ago

Hello Everyone @PyvesB

I hope the villager trade category can be updated. I'll try and explain the best i can.

So any of these trades shown in the below image, will take your trade /aach up by 1, each time you trade the villager image

But if you use the types of trades that gives the player more than 1 item, for example here. 1 emerald = 4 glass. The glass will make your /aach go up by 4! even though you only traded once with the villager. image

It turns out some players have been using this technique to unlock my trading achievements really quickly which seems cheating. Take a look at this. the /aach goes up by 16 every time you trade the villager which i dont think is right. 1 trade should be 1 /aach. not 16! image

Thank you

PyvesB commented 3 years ago

Hello @rosiejane ! 👋🏻

Yes, I think that makes sense, thank for reporting. Fixed in aforementioned commit. 😉