refactor(Import de masse): basculer la logique de documentation (schema -> table) dans un composant dédié #4854
Conversation
17ad7c5 to
be53085
Compare
|
|
||
| <script> | ||
| const schemaTypes = { | ||
| date: "Date (au format AAAA-MM-JJ)", |
There was a problem hiding this comment.
| date: "Date (au format AAAA-MM-JJ)", | |
| date: "Date au format AAAA-MM-JJ", |
There was a problem hiding this comment.
Je trouve que les parenthèses compliquent la lecture
There was a problem hiding this comment.
j'ai opté pour garder les parenthèses un peu partout (y'en avait déjà, et pour homogénéiser, on pourra décider d'homogénéiser dans l'autre sens si besoin)
| number: "Chiffre", | ||
| siret: "14 chiffres, avec ou sans espaces", | ||
| string: "Texte", | ||
| string_enum: "Texte (choix unique)", |
There was a problem hiding this comment.
| string_enum: "Texte (choix unique)", | |
| string_enum: "Texte choix unique", |
| siret: "14 chiffres, avec ou sans espaces", | ||
| string: "Texte", | ||
| string_enum: "Texte (choix unique)", | ||
| string_enum_multiple: "Texte (choix multiples)", |
There was a problem hiding this comment.
| string_enum_multiple: "Texte (choix multiples)", | |
| string_enum_multiple: "Texte choix multiples", |
| <td>{{ field.name }}</td> | ||
| <td>{{ field.title }}</td> | ||
| <td> | ||
| <p>{{ field.description }}</p> |
There was a problem hiding this comment.
La description n'est pas présente sur tous les champs, ce qui fait qu'on affiche quand même un paragraphe vide. Ce paragraphe a aussi un margin-bottom ce qui créé des espacements en "top" inégaux dans toute la colonne
| <p v-if="field.constraints && field.constraints.enum_multiple"> | ||
| Spécifiez plusieurs options en séparant avec un | ||
| <code>{{ field.constraints.enum_multiple_seperator }}</code> | ||
| . |
There was a problem hiding this comment.
Je pense que le "." n'est pas nécessaire en bout de phrase, en plus comme il est proche de la une virgule utilisée en séparateur, ça peut porter à confusion.
…ma -> table) dans un composant dédié
8d50f13 to
ac2bfa5
Compare
…ma -> table) dans un composant dédié (#4854)
Quoi
Suite à #4832, je bascule toute la logique dans un nouveau composant
SchemaTable.vueUtile pour réutiliser dans la/les PR à venir (à commencer par #4853)