Skip to content

v2.0.0#1

Open
totemo wants to merge 38 commits intoredwallhp:masterfrom
NerdNu:master
Open

v2.0.0#1
totemo wants to merge 38 commits intoredwallhp:masterfrom
NerdNu:master

Conversation

@totemo
Copy link

@totemo totemo commented Apr 8, 2019

totemo added 17 commits April 6, 2019 18:41
 * Make it compile.
 * Ignore Eclipse project files.
Finesse around a probable PaperSpigot bug to ensure that
limit does not get set to 0 instead.
Fix error handling for extra arguments.
User-facing commands that dealt in "careers" now operate on "professions"
instead.
Later, we may add `/vtrade` subcommands as subcommands of `/villager` too.
We may also add `/villager experience` to set the trading experience of the
villager. Let's avoid a future name clash.
The code looks right. I tried setting max uses on each recipe but that
didn't help. Looks like it is not possible to refresh trades via the
Bukkit API. So, for now, the command is not enabled.
@totemo totemo changed the title v1.2.0 v2.0.0 Jul 15, 2019
@totemo
Copy link
Author

totemo commented Jul 15, 2019

This commit combines the v1.2.0 (1.13.2) and v2.0.0 (1.4.3) changes.

If you're not interested in maintaining this repo any longer @redwallhp, would you consider assigning it to the NerdNu organisation?

totemo and others added 11 commits April 3, 2020 15:48
Extend trade editing support to wandering traders.
Document limitations and suggested preparations.
Prevent villagers from being named when the player does not
have build permission at the villager's location.
HOTV disabler added, some dependencies updated
spectreOxion and others added 10 commits October 12, 2025 20:26
`OldEnum#toString` no longer returns a simple key representation in 1.21.10, breaking command feedback for `/villager spawn` and `/villager spawnfile`. This also broke serialization to file.

This replaces `OldEnum#toString` with `getKey` followed by `NamespacedKey#getKey` to get the unscoped key, matching previous behavior for both 1.21.8 and 1.21.10.
A villager's profession will reset if they do not have a job site, have 0 XP, and level <= 1.

To avoid this scenario, we will default level to 5. If level 1 is explicitly requested, we employ a workaround setting XP to 1.
Fix villagers losing profession immediately after `/villager spawn`.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants