Rechercher simultanément les sujets publics et les MP

Suite à la discussion de Créer/Voir et Créer les permissions (à nouveau) :

Je pense que c’est une excellente idée. Pour le moment, nous avons le modificateur in:private pour rechercher les MP, alors peut-être quelque chose de similaire comme in:all ou in:everywhere ?

9 « J'aime »

@david : Merci d’avoir créé ce fil (je comptais attendre d’avoir un peu de code :slight_smile: ). J’ai bidouillé un peu ça entre les réunions hier et ce matin, et j’ai une preuve de concept qui ajoute un tag de recherche in:all : GitHub - adqm/discourse at search_all · GitHub

Quelques points à réfléchir :

  • C’est juste un hack rapide, et je n’ai vraiment testé que sur une petite instance de test avec quelques publications.
  • Est-ce que in:all est le bon nom ?
  • Ce serait peut-être bien de pouvoir voir dans la liste quels résultats sont des MP et lesquels sont des sujets.
  • Il faudrait une case à cocher dans la fenêtre de « recherche avancée ».

Je suis ravi de continuer à bidouiller quand j’aurai du temps. Et je ne connais pas encore vraiment Ruby, Rails ni la base de code de Discourse, donc tous les retours et critiques constructives sont les bienvenus !

5 « J'aime »

Super ! N’hésitez pas à ouvrir une PR dès que vous êtes prêt, ainsi les gens pourront commenter plus facilement.

Je pense que c’est correct, sauf si quelqu’un a de meilleures idées ? cc @codinghorror ?

Ce serait effectivement bien, mais je ne m’inquiéterais pas de cela pour commencer. Cela peut faire l’objet d’une PR ultérieure.

Encore une fois, ce serait agréable, mais ce n’est pas nécessairement à faire en même temps. De toute façon, c’est une fonctionnalité plutôt destinée aux utilisateurs avancés.

6 « J'aime »

Après avoir un peu examiné, il s’avère que ce n’est pas tout à fait prêt pour le moment… mais je continuerai à y travailler, et je soumettrai une PR une fois que j’aurai corrigé les bugs que je connais.

7 « J'aime »

Je viens d’ouvrir une PR. Vos réflexions, retours et critiques sont les bienvenus !

7 « J'aime »

in:all convient, in:everything est trop long à taper à mon avis.

8 « J'aime »

Grâce à @hartz, cela est maintenant implémenté :confetti_ball:

Vous pouvez utiliser in:all pour rechercher vos MP et sujets publics au même endroit. Par exemple :

11 « J'aime »

Hmm. Les résultats de la messagerie privée (MP) ne devraient-ils pas afficher l’enveloppe MP à côté du résultat ? Sinon, comment savoir lequel est lequel ?

12 « J'aime »

Oui, ce serait bien. @hartz l’a d’ailleurs déjà suggéré, mais j’ai insisté pour intégrer d’abord la fonctionnalité de base.

@hartz, aimerais-tu essayer de rendre cela fonctionnel dans une deuxième PR ? Sinon, je peux m’en charger.

8 « J'aime »

Bien sûr, je suis ravi d’essayer. J’ai envie d’explorer un peu plus la base de code (puisque je ne suis pas encore tout à fait familier avec l’organisation), et ce serait un bon moyen de le faire.

Cela prendra probablement plusieurs jours avant que je puisse vraiment m’y pencher, donc si tu veux intervenir et t’en occuper, c’est aussi très bien.

6 « J'aime »

@david : J’ai enfin pu jeter un œil à cela (différencier l’affichage des MP et des sujets dans les résultats de recherche), et j’ai mis en place localement une solution de contournement peu élégante qui semble fonctionner, mais qui nécessite probablement pas mal d’affinage. Dois-je ouvrir un nouveau sujet dans la catégorie dev ou quelque chose de similaire si je pense que ce n’est pas encore tout à fait prêt, ou dois-je simplement ouvrir une PR et l’utiliser comme espace de discussion ?

4 « J'aime »

Si vous avez quelque chose de fonctionnel, une PR serait excellente — cela facilite les commentaires sur des parties spécifiques du code. Si vous le souhaitez, GitHub propose une nouvelle fonctionnalité « PR brouillon » (cliquez sur la flèche à côté de « créer une pull request » avant de la soumettre).

7 « J'aime »

D’accord, ça me va, merci ! Je viens de soumettre une PR.

7 « J'aime »

L’icône est maintenant fusionnée — merci @hartz :slight_smile:

8 « J'aime »

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