CoreNetwork / TradeCraft

Custom villager tiers, trades and rebalancing. [No longer updated]
MIT License
2 stars 6 forks source link

Trades are not working with latest source #40

Closed Kongolan closed 9 years ago

Kongolan commented 9 years ago

Compiled the latest source and can't get any trades to work, the careers work as expected.

What did I do:

Config.yml: http://pastebin.com/7eEQjA4x

Log: http://pastebin.com/P6i0mhZq

Screenshot: http://fs1.directupload.net/images/141206/9cn5i72t.png

matejdro commented 9 years ago

I think that should fix it, try it out with latest source.

eliadil commented 9 years ago

Hi ! Other than using the code with fix that matejdro posted, you also need to change one node in your config Currency: 388 to Currency: emerald

We are moving away from using the item ids to their names - you can still use id: <number in defining trades (for now).

Also, the enchant: definition changed in config - please refer to the new version of https://github.com/CoreNetwork/TradeCraft/blob/master/resources/example.yml to see how its done.

Kongolan commented 9 years ago

Ohh that is the problem, thanks!