Facilite la création / assignation d'un article à un workspace #1140
Conversation
✅ Deploy Preview for stylo-docs canceled.
|
✅ Deploy Preview for stylo-dev ready!
To edit notification comments on pull requests, go to your Netlify site configuration. |
|
Les conflits viennent de a60e923, j'ai pas le courage de rebase… |
|
Je viens de rebase la branche ✨ |
|
J'ai lancé le linter sur chaque commits afin d'avoir le moins de changements possibles |
|
Yeah merci beaucoup 🙏🏻 |
|
J'ai peut être cassé quelque chose lors du rebase, sur la preview Netlify ça me dit D'un point de vue graphique, je pense que c'est important que les étiquettes aient le même affichage (bordure + pastille de couleur). On pourrait imaginer un design un peu différent pour l'objet "Workspace" afin de distinguer les deux visuelles. Je laisse @maiwann trancher 😉 ActuellementDétail d'un article
Création d'un article
PropositionDétail d'un articlePas de changement. Création d'un article
(le fond de la checkbox prend la couleur de l'étiquette/de l'espace de travail quand coché) |
Je regarde 👀 |
|
Yeeaaah c'est stylé ! Merci pour cet ajout ! :D |
There was a problem hiding this comment.
Lorsque j'essaie de créer un article depuis un espace de travail (avec ou sans la coche), j'obtiens cette erreur dans une modale en bas à droite de l'écran: Impossible de créer un nouvel article: Error: Unknown type "CreateArticleInput". Did you mean "CreateCorpusInp...
Edit : Idem depuis l'espace personnel.
Oui c'était pour faire un truc un peu plus compact, et éviter d'avoir recours à un composant qui se trouvait être très spécifique (très focalisé "gestion de modification d'état individuelle"). Si on allait vers un équivalent de composant d'étiquette — mutualisable pour les workspace et les tags — pour du formulaire, de l'affichage et/ou filtrage, ça me va aussi. |
💯 |
Il y a des changements côté backend/GraphQL. La preview ne déploie que le frontend et utilise le backend/GraphQL de stylo-dev donc la version n'est pas raccord. |
Ok donc c'est normal que je ne puisse pas créer d'article. On verra ça quand ce sera en preprod du coup ? |
|
Ah oui, j'ai modifié la requête de création ( |
|
En vrai, je suis partant pour ajouter des tests côté backend, et maintenir l'apparence précédente des étiquettes (créer un composant réutilisable). |
|
Je ne trouve pas lisible le fait d'avoir la checkbox de la couleur de l'espace de travail / de l'étiquette, je préfèrerai garder la pastille à coté. Comme les 2 sont bien titrées je trouve OK qu'ils aient un aspect similaire, à voir si ça crée de la confusion chez les utilisateurs. Est-ce que ça répond à ce pour quoi tu m'as ping @ggrossetie ? |
Oui parfait, merci pour ton retour 👍🏻 |
…n Articles et Corpus et Header
On l'oubliait en l'extrayant via du destructuring.
2caee7b to
cd1a6b0
Compare
|
J'ai effectué pas mal de changements pour créer un composant de case à cocher réutilisable (y'en avait 3 différents qui faisaient foncièrement la même chose), notamment pour harmoniser aussi les marges/tailles des différents composants (qui étaient tous stylés un peu différemment, spécifiquement). La mise à jour d'un tag dans la liste d'articles ne donne pas un résultat satisfaisant — ça produit un flash le temps du rechargement des données. Mettre à jour le titre "vide" les tags de toute la page. Je ne suis pas encore au point sur l'usage du flux de données de |
Est-ce que c'est possible d'extraire uniquement ces changement afin d'avancer par étape ? |
|
Oui t'as raison, je me suis grave emballé 😅 |
T'inquiète ça m'arrive souvent aussi ! m2-res_360p.mp4 |
fixes #875
fixes #1065