In reply to goldfishbrigade:

 You do realize this mod is abandoned and won't be receiving updates anymore? 

Please, update for 1.16 fabric <3
Its a very perspective modloader with good optimization and graphics (For example, on forge fps=50, on fabric its 750) (Really)
I think, many developers will soon switch to it

For anyone else having problems editing/ removing existing villager trades to replace them with custom ones; any mod that adds/ edits trades will likely conflict and prevent changing/ removing vanilla trades. You must disable their trade options in the configs. 

Conflicting mods:

Charm.

Quark.

Actually additions.

Hunger overhaul if harvestcraft is installed.

Confirmed to work with editing trades:

Fish undead rising. 

In reply to 3D1ofakind:

 No problem. I always like to post solutions from my hours of bug testing mods so others don't have to go through it. 

In reply to Tenchu98_1:

 The only way to do this is remove existing trades and specify new ones. E.g this would change the vanilla blacksmith to use coins and add items from other mods (the important part here is the clear command at the top that removes all existing trades, the rest can be changed to whatever. this example uses coins from ancient warfare, hence the need for specifying the items nbt data tag):

{ 

      {  Profession: "minecraft:smith",  Career:"armor",

     Offers:{ Recipes:[

{action:"clear"},

{action:"add", buy:{id:"ancientwarfarenpc:coin",Count:{min:12,max:18},tag:{metal:silver}},sell:{id:"minecraft:iron_ingot",Count:6}, CareerLevel:1},

{action:"add", buy:{id:"minecraft:coal",Count:{min:25,max:33}},sell:{id:"ancientwarfarenpc:coin",Count:5,tag:{metal:silver}}, CareerLevel:1,maxUses:5},

{action:"add", buy:{id:"ancientwarfarenpc:coin",Count:3,tag:{metal:silver}},sell:{id:"minecraft:shield",Count:1}, CareerLevel:1}

        ]

    }

}

 Hay is there any way you can create a a config that allows you to increase how often a custom villager spawns. I want to create a custom Villager that spawns in every village.

Hi! I have some question I need to help with..

2. [ 1 ] 
3. I have removed fletcher vanilla trades, and add new trade. below is the current trade I have set for fletcher. Some reason, I can no longer trade with fletcher, until I remove 1 of the remove action. what did I do wrong? can someone help me take a look? I just wanted to remove all fletcher vanilla trades..
   
   

NOTE: I have changed farmer, butcher, blacksmiths trades too with the same exact action with different trades, they works well but fletcher...

{ 
	Profession:"minecraft:farmer",
	Career:"fletcher",

	Offers:{
		Recipes:[
			
			{action:"remove", CareerLevel:1,buy:{id:"minecraft:string"},sell:{id:"minecraft:emerald"}},
			{action:"remove", CareerLevel:1,buy:{id:"minecraft:feather"},sell:{id:"minecraft:emerald"}},
			{action:"remove", CareerLevel:1,buy:{id:"minecraft:flint"},sell:{id:"minecraft:emerald"}},
                        {action:"remove", CareerLevel:1,buy:{id:"minecraft:emerald"},sell:{id:"minecraft:arrow"}},
			{action:"remove", CareerLevel:2,buy:{id:"minecraft:emerald"},sell:{id:"minecraft:bow"}},
			{action:"remove", CareerLevel:2,buyB:{id:"minecraft:emerald"},buy:{id:"minecraft:gravel"},sell:{id:"minecraft:flint"}},
			{action:"remove", CareerLevel:3,buy:{id:"minecraft:emerald"},sell:{id:"minecraft:bow"}},
			
			{action:"add", CareerLevel:1,buyB:{id:"minecraft:feather",Count:64},buy:{id:"minecraft:feather",Count:{min:16,min:64}},sell:{id:"harvestcraft:garliccoinitem",Count:{min:9,max:12}}},
			{action:"add", CareerLevel:1,buy:{id:"minecraft:string",Count:{min:25,max:40}},sell:{id:"harvestcraft:garliccoinitem",Count:{min:5,max:7}}},
			
			{action:"add", CareerLevel:2,buy:{id:"minecraft:diamond",Count:{min:1,max:3}},sell:{id:"minecraft:arrow",Count:{min:4,max:8}}},
			{action:"add", CareerLevel:2,buy:{id:"minecraft:emerald",Count:{min:2,max:4}},sell:{id:"minecraft:bow"}},
			
			{action:"add", CareerLevel:2,buy:{id:"minecraft:flint",Count:{min:25,max:40}},sell:{id:"harvestcraft:garliccoinitem",Count:{min:9,max:12}}},
			
			{action:"add", CareerLevel:3,buy:{id:"minecraft:emerald",Count:{min:13,max:17}},sell:{id:"minecraft:bow",Count:1,tag:{ench:"random"}}},
			
			{action:"add", CareerLevel:4,buy:{id:"minecraft:emerald",Count:{min:3,max:5}},sell:{id:"minecraft:tipped_arrow",Count:{min:4,max:7},tag:{Potion:"random"}}},
			{action:"add", CareerLevel:4,buy:{id:"minecraft:emerald",Count:{min:2,max:3}},sell:{id:"minecraft:spectral_arrow",Count:{min:2,max:2}}}
		]
	}

}

[ 2 ] 

I wanted to add tinker's construct trades in it.. but I couldnt get the item ID is kinda merged... as in, they have the same ID and tag.. the only different is their name.. how should I add them in the trade?

In reply to chiikayuki:

 If your removing all trades, instead of multiple remove commands try:

{action:"clear"},

Ive got that at the top of all trades and it works well. 

If someone figures out how to add trades for enchanted books with random enchantments and explorer maps (monument and mansion), let me know. I am trying to add the old trades for the librarian and cartographer. The Wiki is not of any help... Thanks in advance!

In reply to da_real_steve:

 try this:

{{                action: "add", CareerLevel:1, uses:0, maxUses:2147483647,                buy: {id:"modcurrency:banknote",Count:4,Damage:5}, sell:{id:"minecraft:book",Count:1, tag:{ench:"random"}} },

This is one of the trades im using in my "enchanter". Each level they have another random book. Notice the tag tag:{ench:"random"}. I dont know about explorer maps though.

Hope that helps :-)

do the redstoner and brewer show up naturally?

In reply to Forge_User_05597637:

 Not sure about redstoner, but the brewer has appeared naturally when I've played.