-
Notifications
You must be signed in to change notification settings - Fork 20
Fixed for 1.8 #37
base: master
Are you sure you want to change the base?
Fixed for 1.8 #37
Conversation
The question is: should we really update for 1.8 in light of the EULA changed prohibiting cape mods?
|
Oh? I didn't know about that. After having a quick look it seems as though mod capes are frowned upon but not banned. |
That was at the start of this year. This news just came in June. https://mojang.com/2014/06/lets-talk-server-monetisation/ https://mojang.com/2014/06/lets-talk-server-monetisation/
|
hmm, the only other thing is that the whole eula thing was all about servers specifically and a mod like this has nothing to do with servers. |
But it is still giving capes away for free, and they are really only “useful” on servers..
|
But so does optifine, but the exception is its only visible to people who use the mod / library, |
Has been pushed? |
@mmdanggg2 Any idea why this doesn't work in 1.8.9?
|
I think this is just a problem with the field names, because the fix relies on super hacky reflection stuff it is very hard to get it working. It doesn't help that the field name is different in the dev environment than in the compiled game. Take a look at the version of the fix in this: LINK It should work for 1.8.9 |
Thank you @mmdanggg2, it works. Also, I just understood, you're the author of the Doge mod! By the way, I noticed that you didn't have the right values for 1.9.4 or 1.10.2 yet, and neither do I, so maybe we should look at this class by @aidancbrady (notice which branch you're looking at), I hope it helps. Edit: I used the code from your 1.9.4 branch and the value from the class above and it works on my 1.10.2 client! Edit 2: The same code works for 1.9.4 clients, too. Edit 3: It's kind of screwed with the Elytra wings, unfortunately. |
Any idea, @mmdanggg2? |
I've updated it in my mod here |
@Wehavecookies56 And how does this work with the Elytra wings? |
No description provided.