-
-
Notifications
You must be signed in to change notification settings - Fork 2.4k
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
Expand PlayerItemMendEvent #7382
Expand PlayerItemMendEvent #7382
Conversation
ac39622
to
b504a5c
Compare
b504a5c
to
9b5efaf
Compare
Rebased for 1.18.2 Now resolves 2 open issues. |
This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions. |
Now resolves 3 open issues. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I'd write out "Operation" in the getter and setter, even if it does make it pretty long, but otherwise fine
5cd7615
to
97a78c4
Compare
Ok, tested and this fixes the recursion issue, and the new API works as expected. I'll wait for a 2nd approval since this is new API and not a simple fix. |
Any updates on this PR? It seems to be fully implemented and functional already - is there anything that is blocking this from moving forward? |
97a78c4
to
49a0fbc
Compare
39ee9e6
to
a2e44e2
Compare
a2e44e2
to
2da9441
Compare
Resolves #7313
Resolves #7449
Closes #7815
Adds a getter/setter for the operation used to turn the amount of durability repaired, into an XP value to be subtracted away from the total value of the experience orb.