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

Cache les boutons éditer et citer pour les messages des MPs avec une seule personne #6180

Merged
merged 1 commit into from
Oct 3, 2021

Conversation

philippemilink
Copy link
Member

Fix #6104

Cache les boutons Éditer et Citer des messages d'un MP avec une seule personne (conversation avec un bot ou si tous les correspondant ont quitter le MP). En plus ce qui est rapporté dans l'issue, cliquer sur le bouton Citer n'avait en effet aucune action, mais provoquait en plus une erreur JavaScript ! J'ai aussi caché le bouton Éditer, car dans ce cas, l'édition du message est juste un textarea désactivé, sans aucune explication sur pourquoi on ne peut pas éditer le message (j'ai du fouiller dans le code pour voir que c'est désactivé si on est seul dans le MP).

QA

  • Dans un MP où est on tout seul (le plus simple est que user2 envoie un MP à user1, user1 quitte le MP, puis consulter le MP avec user2): il n'y a pas de bouton Citer et Éditer pour le message
  • Dans un MP à plusieurs, les boutons sont bien présents.

@philippemilink philippemilink added the hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest label Oct 1, 2021
@philippemilink philippemilink added this to En développement in Suivi des PR via automation Oct 1, 2021
@philippemilink philippemilink moved this from En développement to En attente de QA in Suivi des PR Oct 1, 2021
@coveralls
Copy link

Coverage Status

Coverage remained the same at 86.621% when pulling 7850bd3 on philippemilink:bouton-citer-mp-seul into 01ee0fa on zestedesavoir:dev.

Copy link
Contributor

@Arnaud-D Arnaud-D 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 ✔️

Suivi des PR automation moved this from En attente de QA to Fusionnable après rebase Oct 3, 2021
@Arnaud-D Arnaud-D merged commit ead125e into zestedesavoir:dev Oct 3, 2021
Suivi des PR automation moved this from Fusionnable après rebase to Fusionnée Oct 3, 2021
@philippemilink philippemilink deleted the bouton-citer-mp-seul branch October 3, 2021 12:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
hacktoberfest-accepted Pull request approuvée pour le Hacktoberfest
Projects
Archived in project
Suivi des PR
  
Fusionnée
Development

Successfully merging this pull request may close these issues.

Bouton "citer" inutile dans un MP avec une seule personne
3 participants