EssentialsX

EssentialsX

2M Downloads

Need to add 'career' type to villagers.

drtshock opened this issue ยท 10 comments

commented
commented

Can you elaborate?

commented

When spawning them

commented

We can add support for >1.8 when this was implemented.

commented

There is no current API, we'll need to provide field mappings for all versions.

commented

Is this still something that needs to be implemented? Because it looks to me as if it has been implemented:

FARMER_VILLAGER("farmer", EntityType.VILLAGER, Villager.Profession.FARMER, true),
LIBRARIAN_VILLAGER("librarian", EntityType.VILLAGER, Villager.Profession.LIBRARIAN, true),
PRIEST_VILLAGER("priest", EntityType.VILLAGER, Villager.Profession.PRIEST, true),
FATHER_VILLAGER("father", EntityType.VILLAGER, Villager.Profession.PRIEST, false),
SMITH_VILLAGER("smith", EntityType.VILLAGER, Villager.Profession.BLACKSMITH, true),
BUTCHER_VILLAGER("butcher", EntityType.VILLAGER, Villager.Profession.BUTCHER, true),

commented

@argetan EssentialsX doesn't include Villager.Career types yet. However, the 1.14 branch now supports these.

commented

So that means this issue can be closed right? Since it has been implemented in the 1.14 branch which, I assume, will be merged into 2.x soon.

commented

I'll close this when 1.14 support is merged.