PyvesB / advanced-achievements

:fireworks: Popular plugin that adds unique and challenging achievements to Minecraft servers.
https://www.spigotmc.org/resources/83466
GNU General Public License v3.0
199 stars 215 forks source link

New settings for jobs reborn hook #903

Closed Teyir closed 3 years ago

Teyir commented 3 years ago

Hello,

Can you add a settings or a new catégorie for the plugin jobs reborn.

New catégorie: Jobs_Total

This new catégorie is for calculate all the level of jobs.

Exemple:

JobsRebornALL: ALL: 10: Goal: Level 10 all jobs! Message: You have reached all jobs level 10! Name: jobsreborn_10_all DisplayName: All level 10 Type: normal

Thank you very much :)

Teyir commented 3 years ago

( PAPI has a placeholder like that → _%jobsr_usertotallevels% )

ps: je viens de voire que vous parlez français, donc on peut communiquer en français si vous le souhaitez ;)

PyvesB commented 3 years ago

Hello/salut @Teyir ! 👋🏻

A la base @n54b a implémenté la catégorie JobsReborn, peut-être que c'est quelque chose qu'il pourra rajouter ? :wink:

Alternativement, résoudre #365 ferait l'affaire également.

ghost commented 3 years ago

C'est aussi quelque chose qui m’intéresse mais ajouté le support de placeholder est très compliqué ... Je ne suis pas assez bon dev pour faire sa ... en skript aucun problème mais en java ...

It's also something I'm interested in but adding placeholder support is very complicated ... I'm not good enough dev to do this ... in skript no problem but in java ...

Teyir commented 3 years ago

Effectivement ajouter le support de PlaceHolderAPI ça serais dingue, on pourrait faire nos propres catégories.

Je pense que vu l’ampleur du plugin il serait possible de demander de l'aider au dev de PAPI ou de d'autres personnes

PyvesB commented 3 years ago

Quelqu'un qui connaissait bien PlaceholderAPI avait commencé une implémentation dans #365, mais il n'y a jamais eu de suite. Un autre problème serait au niveau de l'interface /aach list : tous les succès basés sur PlaceholderAPI feraient partie de la même catégorie, il faudrait #536 pour s'affranchir de cette limitation, et là encore c'est un gros, gros boulot !

Teyir commented 3 years ago

Oui effectivement,

Sinon on créé nos propres catégories, on met un ID etc...

Après c'est plus facile à dire qu'à faire évidement

PyvesB commented 3 years ago

Sinon on créé nos propres catégories, on met un ID etc...

Après c'est plus facile à dire qu'à faire évidement

En parlant de ça, je suis en train de simplifier le code dans l'API pour que ça soit plus facile d'ajouter des catégories sans connaissances en Java. Je passerai un coup de cirage sur la documentation également.

Teyir commented 3 years ago

Super merci beaucoup :)

J'ai vraiment hâte de voire cette nouvelle maj

PyvesB commented 3 years ago

En parlant de ça, je suis en train de simplifier le code dans l'API pour que ça soit plus facile d'ajouter des catégories sans connaissances en Java. Je passerai un coup de cirage sur la documentation également.

C'est fait dans la version 7.0.0. Au programme :

En attendant, je vais fermer ce ticket, mais laisser #365 et #536. 😉