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

Config for agricarnation white list #4446

Closed
wants to merge 3 commits into from
Closed

Conversation

rqy2002
Copy link

@rqy2002 rqy2002 commented Aug 27, 2023

Add config white list for agricarnation, to solve issues like "why agricarnations don't work on something" (such as #3815).

(although we can't judge which block is actually "a crop", players or modpack authors can control it by such a config)

@rqy2002
Copy link
Author

rqy2002 commented Aug 27, 2023

(should we test isValidBonemealTarget then?)

@williewillus
Copy link
Member

why not just a tag?

@rqy2002
Copy link
Author

rqy2002 commented Aug 31, 2023

You are right. I'm just not familier to modding 😆

@TheRealWormbo
Copy link
Collaborator

Is this PR still being worked on?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

3 participants