Skip to content

add entity filter for allay#721

Merged
Alexander01998 merged 2 commits intoWurst-Imperium:masterfrom
DrMaxNix:master
Mar 24, 2023
Merged

add entity filter for allay#721
Alexander01998 merged 2 commits intoWurst-Imperium:masterfrom
DrMaxNix:master

Conversation

@DrMaxNix
Copy link
Contributor

Description

This commit adds an entity filter for allays, as they are currently not included by any other filter.

@CLAassistant
Copy link

CLAassistant commented Oct 19, 2022

CLA assistant check
All committers have signed the CLA.

ThisTestUser added a commit to ThisTestUser/Wurst7 that referenced this pull request Feb 25, 2023
@DrMaxNix
Copy link
Contributor Author

Just out of curiosity: Will this PR be merged some day or am I missing something?

@Alexander01998 Alexander01998 self-assigned this Mar 24, 2023
@Alexander01998 Alexander01998 added type:enhancement New feature or request area:settings Improves the settings of existing features. labels Mar 24, 2023
Copy link
Member

@Alexander01998 Alexander01998 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi, thanks for your contribution. Sorry for the delay in reviewing this PR. I think your code looks good and it makes sense to add an entity filter for allays. I have now tested it locally and it works as expected. I will merge this right away. Thanks again for your interest and patience.

@Alexander01998 Alexander01998 added this to the v7.33 milestone Mar 24, 2023
Alexander01998 added a commit that referenced this pull request Mar 24, 2023
@Alexander01998 Alexander01998 merged commit f471595 into Wurst-Imperium:master Mar 24, 2023
@Alexander01998 Alexander01998 added the status:merged This pull request has been merged, even if GitHub says otherwise. label Mar 24, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area:settings Improves the settings of existing features. status:merged This pull request has been merged, even if GitHub says otherwise. type:enhancement New feature or request

3 participants