K1ngDiZzY / K1ngDiZzY/MineMod

[EPIC] Tradable Interface - Professions

Open
#121 0 comments 0 reactions 1 assignee Claimed by @johnson-darren View on GitHub
documentation enhancement Research
Dominant language
Java
Stars
1
Forks
0
Avg merge
3d 9h
Merged PRs (30d)
12

Description

This will be a Parent Issue with multiple Sub Issues.

Right now, NPCs all must declare their own trades using the Trade Builder, and it is all static.

### Proposed Changes
- Dynamic Professions:
- [ ] Similar to Villagers, NPCs should be able to "Acquire" different professions based on the Profession Block closest to them.
- [ ] NPCs will be "Locked" to a profession when first trade is made.
- [ ] NPCs should have Master Tag "CanTrade" to False by default, and set to True when they Acquire a Profession
- [ ] NPCs should get their trades from a list based on the Profession they Acquired.
- Example: Butcher profession has Food trades, Cartographer profession has Map trades, etc.

### Reasoning
These planned changes will make the Tradable Interface more useful for handling a wider variety of Trading NPCs.

Contributor guide

No contributing guide indexed for this repository

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.