Page pour afficher les infos d'un nouvel ingrédient #1245
+185
−10
Conversation
This file contains 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
hfroot
changed the base branch from
staging
to
declared-ingredient-single-view
alemangui
requested changes
Nov 18, 2024
| <DsfrNotice title="En construction" desc="Des nouvelles fonctionnalités arrivent bientôt !" /> | ||
| <DsfrNotice | ||
| title="NOUVEL INGRÉDIENT" | ||
| desc="Ingrédient non intégré dans la base de donnée et en attente de validation. " |
There was a problem hiding this comment.
Suggested change
| desc="Ingrédient non intégré dans la base de donnée et en attente de validation. " | |
| desc="Ingrédient non intégré dans la base de données et en attente de validation." |
| <p :aria-hidden="true" class="fr-h2">{{ authorizationInfo.flag }}</p> | ||
| <p class="content-center col-span-2">{{ authorizationInfo.text }}</p> | ||
| </div> | ||
| <div v-for="(info, idx) in elementProfile" :key="idx" class="grid grid-cols-3"> |
There was a problem hiding this comment.
Je conseillerais de rendre un peu plus unique l'id de la boucle for. Dans la même page on risque d'avoir d'autres el générés à partir d'un v-for (par ex la liste d'élements de l'autocomplete, etc). Pour éviter des soucis de réactivité côté Vue c'est mieux si chaque boucle v-for génère des ids différents.
Suggested change
| <div v-for="(info, idx) in elementProfile" :key="idx" class="grid grid-cols-3"> | |
| <div v-for="(info, idx) in elementProfile" :key="`element-profile-${idx}`" class="grid grid-cols-3"> |
| import { getTypeIcon, getTypeInFrench, getApiType } from "@/utils/mappings" | ||
| import { handleError } from "@/utils/error-handling" | ||
|
|
||
| const props = defineProps({ type: String, id: Number }) |
There was a problem hiding this comment.
Le paramètre id venant du router n'est pas du type Number mais String :
Invalid prop: type check failed for prop "id". Expected Number with value xxxx, got String with value "xxxx".
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.
Contexte
ETQ Instructrice, je veux voir les détails d'une demande d'un nouvel ingrédient.
Maquette : https://www.figma.com/design/0pv6zTHf7IW3wbxmmScSUh/Nouvel-ingr%C3%A9dient?node-id=13-17906&node-type=frame&t=3Ab8C0Quf57vTLdi-0
Pour tester
Pour tester, detourner une instructrice, et trouver une déclaration avec des nouveaux ingrédients à instruire. Aller regarder la composition. Un bouton devrait être visible à côté de chaque nouvel ingrédient.
Les champs affichés changent selon le type d'ingrédient
À venir
Sur cette page on va avoir, en deuxième temps, les actions à effectuer comme remplacer par un ingrédient existant ou créer un nouvel ingrédient, etc.
En deuxième temps cette page sera aussi accessible depuis le tableau de demandes #1239
Exemplaires