sekwah41 / Re-Skin

Skin customiser for mc
MIT License
7 stars 7 forks source link

Always need OP to set skin #104

Closed nosekefik closed 1 year ago

nosekefik commented 2 years ago

I have the option setSelfSkinsNeedsOp = false but when a non OP player uses /setskin it not works, and appears a message that he need OP. I'm using the latest version on CurseForge (2.5.1) on minecraft 1.18.2 Forge

sekwah41 commented 2 years ago

Are you sure you are on 2.5.1? Im pretty sure we added a fix for that last update.

What is the exact command you are typing?

nosekefik commented 2 years ago

Yes I'm on 2.5.1 I tried with /setskin https://i.imgur.com/mORJxcm.png image image

sekwah41 commented 2 years ago

I'll try to re-create it tomorrow

sekwah41 commented 1 year ago

Just looking back through issues. I think I couldn't recreate it but I'll need to check.

sekwah41 commented 1 year ago

Validated in 1.19.3. Will create a fix.

sekwah41 commented 1 year ago

I will keep an eye on this, I just have been trying to set someone else's skin or something though I have fixed one permission issue I am unsure if this was it.

I know its been a while since the report but could you check if you are still having the issue on the latest version? If you are still having the issue would you be able to send your server.properties and re-skin config?