Skip to content

Add quests custom objectives#455

Merged
AerunDev merged 15 commits intoPokemonWorkshop:developfrom
ralandel:custom-objectives-ralandel
Mar 16, 2025
Merged

Add quests custom objectives#455
AerunDev merged 15 commits intoPokemonWorkshop:developfrom
ralandel:custom-objectives-ralandel

Conversation

@ralandel
Copy link
Contributor

@ralandel ralandel commented Mar 4, 2025

Description

Ajoute les objectifs custom dans PkmnStudio, ces derniers étant validable directement en event-making avec un appel de script.

Note before testing

J'ai mis arbitrairement le CSV 300 pour stocker les textes, mais il va sans dire que pour Studio ça sera de l'ordre de 100XXX plutôt, je vous laisse modifier ça pour choisir celui qui vous convient le plus.

Il faut également que @Palbolsky repasse sur le code "rapidement" pour que l'attribution (et le pointage) vers les textes soient dynamique comme c'est le cas pour les formes de Pokémon par exemple.

Tests to perform

  • Vérifier qu'un Custom Goal est correctement implémenté et traduit dans PSDK.

@AerunDev AerunDev changed the title Custom objectives ralandel Custom objectives Mar 4, 2025
@AerunDev AerunDev added the User story Issues related to a new feature or request label Mar 4, 2025
@AerunDev AerunDev changed the title Custom objectives Add quests custom objectives Mar 4, 2025
@AerunDev
Copy link
Collaborator

AerunDev commented Mar 4, 2025

Merci pour cette actualisation. 😄
Je vais fermer l'ancienne PR.

Comme déjà vu ensemble cependant, cette PR ne respecte pas les patterns mis en place sur Studio jusqu'à présent.

  • On évite autant que faire se peut de mettre des champs de texte avec des index ou des ID alors qu'on a une fonctionnalité spécialement dédiée à l'écriture d'un message, son stockage dans un CSV et sa traduction directement accessible depuis l'input. Si on suit ton implémentation, on oblige l'utilisateur à mémoriser la ligne de son texte et à faire des allers retours sur l'interface de gestion des textes, ce qui est très peu ergonomique.
  • Depuis peu, on a ajouté un composant spécial permettant de gérer des listes de taille raisonnable et d'avoir une valeur "custom" qui permet d'afficher un input derrière. C'est clairement le cas de cette feature donc il peut être utilisé.

On évitera ainsi de modifier le StudioDropDown qui est un composant central, et de faire des manipulations complexes sur les textes alors qu'on a déjà tout ce qu'il faut pour.

J'ai lancé la discussion avec @Palbolsky pour voir ce qu'on peut faire pour améliorer ça.

@AerunDev AerunDev merged commit e4c09cc into PokemonWorkshop:develop Mar 16, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

User story Issues related to a new feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants