Globox1997 / LevelZ

https://modrinth.com/mod/levelz
https://www.curseforge.com/minecraft/mc-mods/levelz
GNU General Public License v3.0
51 stars 50 forks source link

MCA Reborn Integration #181

Open Jurklein opened 2 years ago

Jurklein commented 2 years ago

"Minecraft Comes Alive - Reborn" is a mod which heavily overhauls the villagers. Actually it replaces all standard villagers with new entities, mca.male_villager and mca.female_villager. With these entities the player is able to talk, interact (give simple commands like follow, set profession, gift, etc.) and... trade.

It's an incredibly immersive mod, and I thought it would go nicely with LevelZ.

It seems the VillagerQuests mod already integrates well with MCA Reborn, but LevelZ doesn't currently seem to work; I attest this after an attempted datapack with the following entries under data/levelz/entity/villager_unlock.json:

{
    "replace": true,
    "skill": "trade",
    "level": 1,
    "entity": "minecraft:custom_entity",
    "object": [
        "minecraft:villager",
        "mca:male_villager",
        "mca:female_villager"
    ]
}

Despite the trade skill constraint, I'm able to interact with the mca villager entities at any level.

Is there something that could be done to enable the two mod's integration?

Evidences: Trade Level 0: image

MCA Villager interaction is not blocked: image

PixelatedBruh commented 2 years ago

I would like to add that the trade price is also not affected by the stat for mca villagers.