Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Permet de gérer les auteurs directement depuis la liste des auteurs #6586

Merged
merged 4 commits into from
Mar 3, 2024

Conversation

Arnaud-D
Copy link
Contributor

@Arnaud-D Arnaud-D commented Mar 3, 2024

Retire les deux éléments du menu suivant :

  • Ajouter un auteur
  • Gérer les auteurs

Pour les remplacer par des contrôles intégrés à la liste des auteurs :

  • bouton de suppression sur chaque auteur (sauf si un seul restant) et modale de confirmation
  • retour du bouton "Ajouter un auteur" qui avait sauté à cause d'une faute de frappe dans une refacto.

image

On simplifie ainsi le menu de gauche et d'avoir la gestion des auteurs au plus près de la liste des auteurs.

C'est un sujet qui avait déjà été discuté, mais je n'ai pas pu trouver de ticket correspondant, surprenamment.

Si vous avez des améliorations pour l'aspect et le code CSS, je suis preneur !

Contrôle qualité

Tester la gestion des auteurs

Comparer les versions en ligne, brouillon, en bêta, en validation pour que tout est bon sur les publications.

Non régression : aller voir les autres listes de membres, notamment les MP, la page "Contact", des endroits sur les forums aussi, et les alertes.

@Arnaud-D Arnaud-D added the C-Front Concerne l'interface du site label Mar 3, 2024
@coveralls
Copy link

coveralls commented Mar 3, 2024

Coverage Status

coverage: 88.664%. remained the same
when pulling 67fa84d on Arnaud-D:authors-management
into bcd5efa on zestedesavoir:dev.

Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

La petite suggestion et on est bon.

templates/tutorialv2/includes/headline/authors.part.html Outdated Show resolved Hide resolved
@Arnaud-D Arnaud-D changed the title Permet de gérer les auteurs directement depuis la liste des auteur Permet de gérer les auteurs directement depuis la liste des auteurs Mar 3, 2024
Copy link
Member

@philippemilink philippemilink left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

QA OK ✔️

Plus qu'à faire la même chose pour les contributeurs d'un contenu 😉 !

@philippemilink philippemilink merged commit 57a4c11 into zestedesavoir:dev Mar 3, 2024
12 checks passed
@Arnaud-D Arnaud-D deleted the authors-management branch March 3, 2024 15:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
C-Front Concerne l'interface du site
Projects
Archived in project
Development

Successfully merging this pull request may close these issues.

3 participants