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

Add player attributes from Hexxy Attributes #823

Open
wants to merge 3 commits into
base: main
Choose a base branch
from

Conversation

beholderface
Copy link
Contributor

This code is largely based on that of Hexxy Attributes by justs_js, PR'd with his permission.
https://discord.com/channels/936370934292549712/1011455473528098857/1327275691343085578

attributes for being allowed to staffcast, media consumption, player ambit radius, and sentinel ambit radius
code largely based on justs_js' Hexxy Attributes addon, PR made with his permission
@gamma-delta
Copy link
Collaborator

Can I get a second pair of eyes on this? @object-Object @SamsTheNerd ?

Copy link
Member

@SamsTheNerd SamsTheNerd left a comment

Choose a reason for hiding this comment

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

Have like one tiny maybe nitpick that's probably fine to ignore but otherwise it looks fine to me

@@ -25,6 +26,9 @@ public ItemStaff(Properties pProperties) {

@Override
public InteractionResultHolder<ItemStack> use(Level world, Player player, InteractionHand hand) {
if (player.getAttributeValue(HexAttributes.FEEBLE_MIND) > 0){
Copy link
Member

Choose a reason for hiding this comment

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

Is this fine on client?

Copy link
Member

Choose a reason for hiding this comment

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

Nvm, I think it should be fine since they're marked syncable?

out.add(HexAttributes.FEEBLE_MIND);
out.add(HexAttributes.MEDIA_CONSUMPTION_MODIFIER);
out.add(HexAttributes.AMBIT_RADIUS);
out.add(HexAttributes.SENTINEL_RADIUS);
Copy link
Member

Choose a reason for hiding this comment

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

Should these be like,, auto grabbed out of the HexAttributes instead of manually listed in 3 separate places?

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

Successfully merging this pull request may close these issues.

3 participants