Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

AE2 Fluix Researcher Trades not being removed #15

Closed
Electrolyte220 opened this issue Jun 29, 2024 · 3 comments
Closed

AE2 Fluix Researcher Trades not being removed #15

Electrolyte220 opened this issue Jun 29, 2024 · 3 comments

Comments

@Electrolyte220
Copy link

Minecraft Version

1.20.1

Mod Version

0.10.0

Describe the bug!

I'm trying to remove trades for the AE2 Fluix Researcher, however it does not seem to work at any level. It works with other modded villagers (like the Advanced Peripherals computer scientist), but not with this one.

Script

This is the same code I have for the AP one, but it does not work for AE2.

MoreJSEvents.villagerTrades(event => {
  event.removeModdedTrades(['ae2:fluix_researcher'], 1)
})

Crash Report

No response

Log

No response

Additions and Modifications

Yes

Additions and Modifications Description

AE2 Version: 15.2.4

Did the issue happen in singleplayer or on a server?

Singleplayer

@LLytho
Copy link
Member

LLytho commented Jun 29, 2024

however it does not seem to work at any level.

but it does not work for AE2

On which level it does not work? Your script just shows for level 1.

@Electrolyte220
Copy link
Author

It does not work for any level.

@LLytho
Copy link
Member

LLytho commented Jun 29, 2024

use removeVanillaTrades as AE2 uses vanilla trade objects internally.

@LLytho LLytho closed this as completed Jun 29, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants