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

HD Skin Using Incorrect Body Type #80

Closed
SkyeShey7 opened this issue Dec 29, 2024 · 7 comments · Fixed by #81
Closed

HD Skin Using Incorrect Body Type #80

SkyeShey7 opened this issue Dec 29, 2024 · 7 comments · Fixed by #81
Assignees

Comments

@SkyeShey7
Copy link

SkyeShey7 commented Dec 29, 2024

I am trying to import a HD skin that is 128x128 but it won't let me use the slim body type. What do I need to do? I am using 1.21.1 for my modpack.
Screenshot 2024-12-29 at 1 06 20 AM

@Sollace
Copy link
Member

Sollace commented Dec 29, 2024

What version of the mod are you using?

@CassieRoseZA
Copy link

Image

@Sollace
Copy link
Member

Sollace commented Jan 25, 2025

I was able to determine this is a problem with the skin server and not the mod. The model type is not being saved correctly.

Forwarding this to @mattmess1221

@Sollace
Copy link
Member

Sollace commented Jan 25, 2025

Posted form:
Image
Server response after:

Image

Raw JSON data:
message-1.txt

This is unfortunately something I cannot fix without @mattmess1221 's assistance

@mattmess1221
Copy link
Member

mattmess1221 commented Jan 25, 2025

In v1 of the API, model should be passed as json to the meta form part.

See: https://skins.minelittlepony-mod.com/docs#/Texture%20Uploads/put_texture_api_v1_textures_put

@CassieRoseZA
Copy link

i can also confirm its effecting all 1.21.1 versions

@mattmess1221
Copy link
Member

#81 should fix this, though I haven't tested it thoroughly.

@mattmess1221 mattmess1221 self-assigned this Jan 25, 2025
@Sollace Sollace closed this as completed in a58e2fe Feb 9, 2025
@Sollace Sollace reopened this Feb 9, 2025
@Sollace Sollace closed this as completed Feb 9, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging a pull request may close this issue.

4 participants