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

Ideas for future versions of the plugin #89

Closed
6 of 15 tasks
xknat opened this issue Aug 2, 2018 · 12 comments
Closed
6 of 15 tasks

Ideas for future versions of the plugin #89

xknat opened this issue Aug 2, 2018 · 12 comments

Comments

@xknat
Copy link
Member

@xknat xknat commented Aug 2, 2018

High priority:

  • > Java 8 support
  • skinsrestorer > changeskin

why would we disable if changeskin is there? we should do this both ways and not be the underdogg

  • Restore SkinsRestorerAPI.java

After the huge changes by DonotSpamPls the api stopped working

  • Make the config system work reliably [partly done 13.6]

Medium priority:

  • forcebungeemode, because some "cant" do that normal
  • /skins (gui) for bungee
  • Rework /skins (gui) to work on both -1.12 and +1.13 (not going to happen)
  • skinsrestorer.skin.own will allow to set your own skin if PER_SKIN_PERMISSIONS = True
  • option to prevent 14:02:58 [INFO] [SkinsRestorer] Switching to Mojang to get UUID. (xknat) spam: https://pastebin.com/CddWcsSY

Low priority:

  • Dont set cooldown to 10-20s when skin failed
  • Add a debug option that gives more insight on what causes a problem.
  • Implement HikariCP
  • Replace Reflection with Abstraction (-> makes no sense because thats similar to a full rewrite)
  • a way to blacklist a spigot server if using bungee to avoid bot attacks
  • Add more languages to the plugin (French, German, etc.)

Use preset langues files within the plugin.

@Raideerke
Copy link

@Raideerke Raideerke commented May 20, 2019

Hey. We're still waiting for the API, is there any plans to implement it in the near future?

@xknat
Copy link
Member Author

@xknat xknat commented May 20, 2019

@Raideerke sadly we don't have such experience to code a api into the plugin. so help is still wanted from a dev that can perform such things and look into our old api

@McLive
Copy link
Contributor

@McLive McLive commented May 20, 2019

Hey. We're still waiting for the API, is there any plans to implement it in the near future?

It will be included in one of the next releases. Currently there is very little time because of other projects so there is no ETA, yet!

@Matocolotoe
Copy link

@Matocolotoe Matocolotoe commented Jun 7, 2019

  • Rework /skins (gui) to work on both -1.12 and +1.13 (not going to happen)

The error is just about the ID of the white stained glass pane not working so why
At least if the command isn't meant to work anymore on -1.12 just remove it

@McLive
Copy link
Contributor

@McLive McLive commented Jun 7, 2019

Why should we remove it if it works fine on newer versions?

@Matocolotoe
Copy link

@Matocolotoe Matocolotoe commented Jun 7, 2019

It takes 2 simple code changes to make it work on older versions, which are still used a lot
I actually wanted to say that if the command isn't meant to be working on 1.12 and lower remove it on these versions

@McLive
Copy link
Contributor

@McLive McLive commented Jun 7, 2019

Please do a pull request to help us with that

@Samuelva
Copy link

@Samuelva Samuelva commented Jun 9, 2019

Is this also being considered or even possible at all?

@xknat
Copy link
Member Author

@xknat xknat commented Jun 16, 2019

@Samuelva i think you can best check with https://github.com/riflowth/SkinSystem for that!

@McLive
Copy link
Contributor

@McLive McLive commented Jul 20, 2019

@McLive McLive closed this Jul 20, 2019
Enhancements for future versions automation moved this from To do to Done Jul 20, 2019
@qba567
Copy link

@qba567 qba567 commented May 18, 2020

Please add PlaceholderApi support

@SrTeamAdmin
Copy link
Contributor

@SrTeamAdmin SrTeamAdmin commented May 18, 2020

@qba567 what placeholder do you need?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Linked pull requests

Successfully merging a pull request may close this issue.

None yet
8 participants