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

Règle un problème d'affichage des votes dans les MPs #6375

Merged
merged 2 commits into from
Aug 12, 2022

Conversation

Migwel
Copy link
Contributor

@Migwel Migwel commented Aug 12, 2022

Fix #6372 (partiellement)

L'issue 6372 reportait deux problèmes indépendants concernant les votes dans les MPs. J'ai décidé de les régler avec deux PR indépendantes. L'autre est dans #6374.

Contrôle qualité

  • Se connecter avec user1
  • Envoyer un MP à user2
  • Se connecter avec user2
  • Ouvrir le MP envoyé par user1 et liker le premier message
  • Rafraichir la page du MP
    Comportement actuel: Le pouce en l'air est vert "clair" (avec une opacité de 0.5)
    Nouveau comportement: Le pouce en l'air est vert foncté (avec un opacité de 1)

@coveralls
Copy link

coveralls commented Aug 12, 2022

Coverage Status

Coverage increased (+0.002%) to 88.19% when pulling d27c5e2 on Migwel:issue6372b into b597425 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 ✔️

@Arnaud-D Arnaud-D enabled auto-merge (squash) August 12, 2022 15:14
@Arnaud-D Arnaud-D merged commit b6576ca into zestedesavoir:dev Aug 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Dans les messages privés, les votes sont anonymes
3 participants