Pull Request : SiteSetting aux utilisateurs anonymes pour "aimer" les publications

Salut à tous,

J’ai ouvert une PR pour permettre aux utilisateurs anonymes d’aimer les publications. C’est un réglage du site, donc les administrateurs ont la possibilité d’activer ou de désactiver la fonctionnalité.

Pourriez-vous s’il vous plaît me faire une revue de code ? :grinning_face_with_smiling_eyes:

Contexte : Je fais partie d’une organisation 501(c)3 qui utilise Discourse, où nos comptes sont enregistrés sous nos vrais noms. Nous encourageons la discussion anonyme, car certaines personnes veulent partager des expériences personnelles et des sujets sensibles (santé mentale, souvenirs difficiles du passé, etc.). La discussion anonyme a été extrêmement riche et utile, et l’ajout d’une fonctionnalité d’« j’aime » anonyme aiderait à évaluer l’interaction de la communauté sur des sujets où les gens pourraient fortement sympathiser ou être d’accord, mais ne pas se sentir à l’aise de mettre leur nom en avant.

Merci !!

10 « J'aime »

Quelqu’un pourrait-il m’indiquer le processus pour obtenir l’examen et l’approbation du mainteneur ? :smiley:

Merci !

2 « J'aime »

Merci pour la PR et ce sujet @meltingmettle_gh, un membre de l’équipe examinera la PR sous peu.

4 « J'aime »

Salut, nous avons examiné et laissé quelques commentaires sur les traductions et l’expérience utilisateur.

4 « J'aime »

Bonjour ! J’ai mis à jour la PR et elle est prête pour une nouvelle révision. Merci !

3 « J'aime »

Revu – nécessite plus de travail. Merci de m’avoir signalé ici !

3 « J'aime »

Puis-je avoir une astuce rapide sur l’endroit où invoquer l’icône de refus ? J’ai regardé à nouveau mais je n’arrive pas à comprendre. :sweat_smile:

Du commentaire GH :

Salut @nattsw, merci pour la deuxième relecture. J’ai un peu de mal à comprendre où le refus est censé se produire puisque post_undo_action_window_mins n’est référencé que dans can_delete_post_action?, et la méthode renvoie correctement false une fois que post_undo_action_window_mins est dépassé.

Cependant, malgré le fait que cette méthode renvoie correctement false une fois la fenêtre fermée, le symbole de refus n’apparaît pas, mais il n’y a pas d’autre méthode qui référence post_undo_action_window_mins. Je me rends compte que can_delete_post_action? est appelée au chargement de la page, mais je n’arrive pas à localiser où l’icône de refus réelle est invoquée.

Pourriez-vous m’indiquer le bloc de code qui appelle le symbole de refus ? Merci !

2 « J'aime »

Merci pour le conseil ! C’était super utile ! J’ai appliqué un correctif, vérifié manuellement le comportement et ajouté une couverture de test supplémentaire. C’est prêt pour une autre session de révision quand vous aurez le temps. :smiley:

5 « J'aime »

Merci beaucoup pour cette fonctionnalité Emmett, bons tests ! Nous attendons avec impatience vos futures contributions.

:git_merged:

et désolé pour le délai des revues !

7 « J'aime »

Ce sujet a été automatiquement fermé après 4 jours. Les nouvelles réponses ne sont plus autorisées.