Öffentliche Themen und PMs gleichzeitig durchsuchen

Fortsetzung der Diskussion aus Erstellen/Anzeigen und Erstellen von Berechtigungen (nochmal):

[quote=“hartz, post:100, topic:33150”]
es wäre fantastisch, wenn es eine Möglichkeit gäbe, schnell die Frage zu beantworten: „Welche Threads muss ich gerade möglicherweise beantworten?

9 „Gefällt mir“

@david: Danke, dass du diesen Thread erstellt hast (ich wollte eigentlich warten, bis ich etwas Code habe :)). Ich habe gestern und heute Morgen zwischen den Meetings ein wenig daran gearbeitet und habe einen Proof of Concept, der den Suchtag in:all hinzufügt: GitHub - adqm/discourse at search_all · GitHub

Ein paar Dinge, über die man nachdenken sollte:

  • Das ist nur ein schneller Hack, und ich habe es außer mit einer kleinen Testinstanz mit ein paar Beiträgen nicht wirklich getestet.
  • Ist in:all der richtige Name?
  • Es wäre schön, in der Liste sehen zu können, welche Ergebnisse PMs und welche Themen sind.
  • Es braucht ein Kontrollkästchen im Fenster für die “erweiterte Suche”.

Ich bin gerne bereit, weiter daran zu arbeiten, wenn ich Zeit habe. Und ich kenne Ruby, Rails oder die Discourse-Codebasis noch gar nicht, also sind jedes Feedback und konstruktive Kritik definitiv willkommen!

5 „Gefällt mir“

Toll! Bitte eröffne eine PR, sobald du soweit bist, damit andere leichter Kommentare abgeben können.

Ich denke, das ist in Ordnung, es sei denn, jemand hat bessere Ideen? cc @codinghorror?

Das wäre gut, aber ich würde mir dafür am Anfang keine Sorgen machen. Das kann später in einer separaten PR umgesetzt werden.

Auch das wäre schön, muss aber nicht unbedingt gleichzeitig erledigt werden. Diese Funktion ist ohnehin eher auf Power-User ausgerichtet.

6 „Gefällt mir“

Nach etwas mehr Herumprobieren stellt sich heraus, dass es noch nicht ganz fertig ist … aber ich werde weiter daran arbeiten und einen PR einreichen, sobald ich die mir bekannten Bugs behoben habe.

7 „Gefällt mir“

Gerade einen PR eröffnet. Freue mich über Gedanken, Feedback und Kritik!

7 „Gefällt mir“

in:all ist okay, in:everything ist meiner Meinung nach zu viel Tipparbeit.

8 „Gefällt mir“

Dank @hartz ist dies jetzt umgesetzt :confetti_ball:

Du kannst in:all verwenden, um deine PMs und öffentlichen Themen an einem Ort zu durchsuchen. Zum Beispiel:

11 „Gefällt mir“

Hmm. Sollten PM-Ergebnisse nicht den PM-Umschlag neben dem Ergebnis haben? Sonst weiß man ja nicht, welches zu welchem gehört.

12 „Gefällt mir“

Ja, das wäre gut. @hartz hat das eigentlich schon vorgeschlagen, aber ich habe darauf gedrängt, zuerst die grundlegende Funktion einzubauen.

@hartz, möchtest du versuchen, das in einem zweiten PR umzusetzen? Wenn nicht, kann ich mir das ansehen.

8 „Gefällt mir“

Klar, ich probiere das gerne aus. Ich möchte mich noch etwas mehr im Codebase umschauen (da ich mir noch nicht ganz sicher bin, wie die Dinge dort liegen), und das wäre ein guter Weg, das zu tun.

Es wird aber wahrscheinlich noch einige Tage dauern, bis ich mir das wirklich genauer ansehen kann. Wenn du also lieber selbst reinspringen und es erledigen möchtest, ist das auch völlig in Ordnung.

6 „Gefällt mir“

@david: Ich hatte endlich Gelegenheit, mir das anzusehen (die Unterscheidung der Anzeige von PMs und Themen in den Suchergebnissen). Ich habe lokal einen wenig eleganten Hack implementiert, der zwar zu funktionieren scheint, aber wahrscheinlich noch einiges an Verfeinerung benötigt. Soll ich ein separates Thema in der Kategorie „dev

4 „Gefällt mir“

Wenn du bereits etwas Funktionierendes hast, wäre ein PR großartig – das erleichtert das Kommentieren spezifischer Code-Teile. Falls du möchtest, bietet GitHub eine neue „Entwurf-PR“-Funktion (klicke vor dem Einreichen auf den Pfeil neben „Pull Request erstellen“).

7 „Gefällt mir“

OK, klingt gut, danke! Ich habe gerade einen PR erstellt.

7 „Gefällt mir“

Das Symbol wurde jetzt zusammengeführt – danke @hartz :slight_smile:

8 „Gefällt mir“

Dieses Thema wurde automatisch nach 5 Tagen geschlossen. Neue Antworten sind nicht mehr erlaubt.