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

Fix attribute spam for bats #744

Closed
wants to merge 1 commit into from

Conversation

Camotoy
Copy link
Contributor

@Camotoy Camotoy commented Nov 5, 2021

Addresses #728

Since the client doesn't need these attributes, indicate that they shouldn't be sent. This doesn't appear to impact the bat from being rideable.

Feel free to contact me on Discord! Camotoy#5533

Addresses PurpurMC#728

Since the client doesn't need these attributes, indicate that they shouldn't be sent. This doesn't appear to impact the bat from being ridable.
Copy link
Contributor

@encode42 encode42 left a comment

Choose a reason for hiding this comment

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

LGTM, needs a rebase though

Copy link
Member

@theomega24 theomega24 left a comment

Choose a reason for hiding this comment

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

LGTM, I would like a review from @BillyGalbreath

@encode42
Copy link
Contributor

encode42 commented Dec 12, 2021

Re-open this on the 1.18.1 branch, then it is ready to merge

@Camotoy
Copy link
Contributor Author

Camotoy commented Dec 12, 2021

Someone else is free to do that; I'm not sure if I'll get around to it anytime soon.

encode42 added a commit that referenced this pull request Dec 12, 2021
@granny granny closed this Dec 12, 2021
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.

None yet

4 participants