Conversation
2 tasks
…ntries-from-a-select
AerunDev
requested changes
Feb 10, 2025
Collaborator
AerunDev
left a comment
There was a problem hiding this comment.
Hello ! 😄
J'ai pu tester cette PR et globalement rien à redire, c'est très cool, bien joué.
Je note ici quelques régressions qu'il faudrait tout de même qu'on traite avant de la merge. Notamment un crash dans l'input de l'IA personnalisée des dresseurs.
Activation des groupes
L'interrupteur ne devrait pas pouvoir être négatif ou zéro
- ETANT DONNE QUE je souhaite modifier l'interrupteur d'activation d'un groupe de Pokémon sauvages,
- QUAND je sélectionne "Personnalisé" dans la liste déroulante "Activation",
- ET que je rentre un nombre dans le champ de texte "Interrupteur",
- ALORS je peux rentrer un nombre négatif.
Note
Il n'était pas possible de le faire sur la version en prod.
IA custom pour un dresseur
Le texte affiché est en anglais pour la valeur "Custom"
- ETANT DONNE QUE je souhaite préciser une IA personnalisée pour un dresseur
- QUAND je clique sur la liste déroulante "Niveau de l'IA"
- ALORS le texte "Custom" est affiché en dernière position au lieu de "Personnalisé"
- ET le texte "Custom level" est affiché pour l'input au lieu de "Niveau personnalisé"
Crash lorsqu'on ne rentre pas de valeur pour le "Niveau personnalisé" d'une IA
- ETANT DONNE QUE je souhaite préciser une IA personnalisée pour un dresseur
- QUAND je clique sur la liste déroulante "Niveau de l'IA"
- ET que je sélectionne l'entrée "Custom"
- ET que je vide le contenu du champ de texte "Custom level"
- OU que je précise un nombre négatif
- ALORS un crash apparaît avec le log suivant

(Erreur 1 et 2 sur 3)

(Erreur 3 sur 3)
Tip
Peut-être afficher l'astérisque pour mentionner l'obligation de valeur également ?
Procédure d'une capacité
Manque de feedback sur la fonction personnalisée qui ne peut être vide
- ETANT DONNE QUE je souhaite spécifier une fonction personnalisée comme Procédure d'une capacité
- QUAND je sélectionne la liste déroulante "Procédure" pour choisir "Attaque personnalisée"
- ET que je vide le contenu du champ de texte "Fonction"
- ALORS je ne peux pas quitter la modale d'édition si je clique en dehors
- ET je n'ai pas de feedback m'indiquant le caractère obligatoire du champ "Fonction" (astérisque rouge présente en prod) pour mieux comprendre le blocage
Collaborator
Author
|
Et hop, c'est fix 😄 |
AerunDev
approved these changes
Feb 10, 2025
Collaborator
AerunDev
left a comment
There was a problem hiding this comment.
Effectivement c'est nickel à présent, bien joué ! 🎉
Palbolsky
approved these changes
Feb 11, 2025
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Description
closes #401
Added a component that regroups a Select with a custom option and an Input that is displayed only if the custom option is selected. This allows the user to set a value of their choice to the related entity field. Some places where this operation was already implemented have been updated with the new component.
closes #380
The user can now set a custom value to the AI level of a trainer.
Tests to perform
Go to the Groups database page, following tests should pass in both creation and edition of a groupGroup Activation
Editor testing
Input testing
Entity testing
Following tests are only related to edition
Trainer AI
Go to the Trainers database page, following tests should pass in both creation and edition of a trainer
Editor testing
Input testing
Entity testing
Following tests are only related to edition
Move Procedure
Go to the Moves database page and open the 'Setting' Editor
Editor testing
Input testing
Entity testing
Following tests are only related to edition