Cerca simultaneamente argomenti pubblici e PM

Continuando la discussione da Creare/Vedere e Creare i Permessi (di nuovo):

Penso che sia un’ottima idea. Al momento abbiamo il modificatore in:private per cercare i messaggi privati, quindi forse qualcosa di simile come in:all o in:everywhere?

9 Mi Piace

@david: Grazie per aver creato questo thread (stavo pensando di aspettare finché non avessi avuto del codice :slight_smile: ). Ho lavorato un po’ su questo tra una riunione e l’altra ieri e questa mattina, e ho un proof of concept che aggiunge un tag di ricerca in:all: GitHub - adqm/discourse at search_all · GitHub

Alcune cose su cui vale la pena riflettere:

  • Questo è solo un rapido hack e non l’ho testato davvero, se non con una piccola istanza di prova con un paio di post.
  • in:all è il nome giusto?
  • Sarebbe bello poter vedere nell’elenco quali dei risultati sono PM e quali sono topic.
  • Serve una casella di controllo nella finestra “ricerca avanzata”.

Sono felice di continuare a lavorarci quando ho tempo. E non conosco affatto Ruby, Rails o il codice di Discourse, quindi qualsiasi feedback e critica costruttiva è sicuramente benvenuta!

5 Mi Piace

Fantastico! Apri pure una PR quando sei pronto, così le persone potranno commentare più facilmente.

Penso vada bene, a meno che qualcuno non abbia idee migliori? cc @codinghorror?

Sarebbe utile, ma non preoccupiamocene per ora. Potrà essere oggetto di una PR successiva.

Anche questo sarebbe bello, ma non è necessariamente qualcosa da fare allo stesso tempo. Del resto, è una funzionalità pensata principalmente per utenti esperti.

6 Mi Piace

Dopo averci dato un’occhiata più da vicino, sembra che non sia ancora pronto… ma continuerò a lavorarci e invierò una PR una volta risolti i bug che conosco.

7 Mi Piace

Ho appena aperto una PR. Sono felice di ricevere opinioni, feedback e critiche!

7 Mi Piace

in:all va bene, in:everything è troppo lungo da scrivere a mio parere.

8 Mi Piace

Grazie a @hartz, ora è stato implementato :confetti_ball:

Puoi usare in:all per cercare i tuoi messaggi privati e gli argomenti pubblici in un unico posto. Ad esempio:

11 Mi Piace

Hmm. I risultati PM non dovrebbero avere la busta PM accanto al risultato? Altrimenti come fai a sapere quale è quale?

12 Mi Piace

Sì, sarebbe una buona idea. @hartz lo aveva già suggerito, ma ho spinto per implementare prima la funzionalità di base.

@hartz, vorresti provare a farlo funzionare in una seconda PR? Altrimenti posso darci un’occhiata io.

8 Mi Piace

Certo, sono felice di provarci. Sono interessato a esplorare un po’ di più la codebase (visto che non sono ancora sicuro di aver ben chiaro il quadro generale), e questo sarebbe un buon modo per farlo.

Ma ci vorranno probabilmente diversi giorni prima di poter dare un’occhiata vera, quindi se vuoi intervenire e occupartene tu, va bene lo stesso.

6 Mi Piace

@david: Ho finalmente avuto modo di dare un’occhiata a questo (differenziare la visualizzazione dei PM e dei topic nei risultati di ricerca) e ho implementato localmente un workaround poco elegante che sembra funzionare, ma che probabilmente ha bisogno di essere notevolmente affinato. Dovrei aprire un nuovo topic nella categoria dev o qualcosa di simile se penso che non sia ancora pronto per il rilascio, oppure dovrei semplicemente aprire una PR e usarla come luogo per la discussione?

4 Mi Piace

Se hai qualcosa di funzionante, una PR sarebbe ottima: rende più facile commentare parti specifiche del codice. Se preferisci, GitHub ha una nuova funzionalità “Draft PR” (clicca sulla freccia accanto a “crea pull request” prima di inviarla).

7 Mi Piace

Ok, sembra tutto a posto, grazie! Ho appena inviato una PR.

7 Mi Piace

L’icona è stata unita - grazie @hartz :slight_smile:

8 Mi Piace

Questo argomento è stato automaticamente chiuso dopo 5 giorni. Non sono più consentite nuove risposte.