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

Fixed tester/contributor cloaks (and some aura fixes) #388

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

Conversation

jjtParadox
Copy link

@jjtParadox jjtParadox commented Nov 15, 2017

Cloaks and auras broke when Mithion's website moved to HTTPS. The aura code was also erroneously using username instead of UUID to do lookups.

Also added EdwinMindcraft and The_Icy_One to the list, cause those two are pretty awesome.

Also the cloak rendering is clipping through players' arms when they move. Haven't yet figured out how to fix that.

UPDATE: Auras are working client side. Networking will be added after PR #391 is merged in.

Added EdwinMindcraft and The_Icy_One to the list
@TheIcyOne
Copy link

Can confirm I am awesome, although that lookup problem is because I wasn't being awesome and just copy/pasted the old code from before UUIDs were prevalent, and bodged till it worked.

@jjtParadox
Copy link
Author

Whelp just realized the aura particle render system had never been ported from 1.7.10. @EdwinMindcraft do you happen to know what needs to be done to get that working?

Networked auras are still broken
@jjtParadox jjtParadox changed the title Fixed tester/contributor cloaks and auras Fixed tester/contributor cloaks Nov 16, 2017
@jjtParadox
Copy link
Author

Auras should be working clientside now, but data is not synced across servers. That'll have to wait until the PR mentioned in the primary post is merged in.

@jjtParadox jjtParadox changed the title Fixed tester/contributor cloaks Fixed tester/contributor cloaks (and some aura fixes) Nov 16, 2017
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.

2 participants