Whisper umschalten 👁

Erledigt. Dies liegt nun beim ursprĂŒnglichen Autor, der dies weiterverfolgen kann.

2 „GefĂ€llt mir“

Danke fĂŒr die Erledigung. :pray: :slight_smile:

3 „GefĂ€llt mir“

@ganncamp
Ich habe das unmittelbare Problem behoben. Es gibt jedoch einen Vorbehalt. Derzeit unterstĂŒtzt das Discourse-Backend nur Staff-Benutzer, um den post_type eines Beitrags Ă€ndern zu können. Daher habe ich das :eye:-Umschaltsymbol so gestaltet, dass es angezeigt wird, wenn ein Benutzer in whispers_allowed_groups ist und auch ein Staff-Mitglied ist. Lassen Sie uns Ihre Gedanken dazu wissen.

3 „GefĂ€llt mir“

Ich schĂ€tze die MĂŒhe. Ich habe eine Lösung gefunden, die die KomplexitĂ€t etwas reduziert. Vielen Dank auch, dass Sie auf die eigentliche Ursache hingewiesen haben.

2 „GefĂ€llt mir“

Das ist fĂŒr uns in Ordnung; nur Staff-Mitglieder können Whispers sehen.

Und
 ich sehe das noch nicht auf meiner Instanz
?

1 „GefĂ€llt mir“

Und jetzt funktioniert es! Danke!

2 „GefĂ€llt mir“

image
Breaking Change in neuer Version von Discourse

5 „GefĂ€llt mir“

Ich bin kĂŒrzlich auf dasselbe Problem gestoßen, also habe ich die Komponente neu geschrieben und einen PR erstellt. Pull requests · fzngagan/discourse-toggle-whisper

@fzngagan Können Sie sich das ansehen?

11 „GefĂ€llt mir“

Ich habe es gerade getestet und es funktioniert einwandfrei. Der Code sieht auch gut aus. Vielen Dank dafĂŒr, das hat mir viel Arbeit erspart. Gemerged :tada:

5 „GefĂ€llt mir“


Es gibt jedoch einige Probleme, wenn man nicht angemeldet ist.

Dies verursacht auch weit mehr als nur 1 Fehler.

3 „GefĂ€llt mir“

Ich habe einen schnellen PR erstellt, um das Problem zu beheben :+1: (@fzngagan)

5 „GefĂ€llt mir“

Gut gemacht! Ich liebe das Programmieren
 ein einfaches ? und das Problem ist gelöst :joy:

3 „GefĂ€llt mir“

Das Umschalten des Whisper-Symbols erscheint auf unserer Website nicht mehr, wir laufen auf 3.4.1. Hat noch jemand dasselbe Problem?

@fzngagan Hinweis auf einen 404-Link hier:

[Zitat=“fzngagan, Beitrag:1, Thema:181616”]
Dies ist besonders nĂŒtzlich fĂŒr Foren, in denen viele GesprĂ€che im FlĂŒsterton stattfinden und ein Mitarbeiter etwas postet und spĂ€ter den Beitrag vor Nicht-Mitarbeitern verbergen oder anzeigen möchte.
[/Zitat]

Hier besteht ein potenzielles Datenleck, da einige Benutzer die geflĂŒsterten BeitrĂ€ge weiterhin sehen können.

Schritte zur Reproduktion:

  1. Ohne zu flĂŒstern, direkt auf einen Benutzer in einem Beitrag/einer Antwort antworten oder ihn zitieren
  2. Den Beitrag in ein FlĂŒstern Ă€ndern
  3. Der @erwĂ€hnte / @zitierte Benutzer kann den geflĂŒsterten Beitrag weiterhin sehen

Ich glaube nicht, dass ich das reproduzieren kann?

Meine Schritt-fĂŒr-Schritt-Anleitung, nur um zu sehen, ob ich einen :derp: mache:

  • Test_One erstellt einen Beitrag, indem er direkt auf Test_Two antwortet
  • FĂŒgt auch ein Zitat von Test_Three und eine @ErwĂ€hnung von Test_Four hinzu
  • Als Admin, schalte diesen Beitrag in ein Wispern um

Die einzigen Personen, die das Wispern sehen können, sind Admin und Test_One (was erwartet wird, da Sie Ihre eigenen Wisper-BeitrÀge sehen können, selbst nachdem Sie keine Wispern mehr erstellen können)

Ist es eine Sache des Neuladens der Seite?

1 „GefĂ€llt mir“

Ich werde mich mit meinem Kollegen, der das Problem gemeldet hat, in Verbindung setzen, um dies zu bestÀtigen, aber:

Ist Test_One ein Administrator? :thinking:

Nein, nur ein normaler Nicht-Mitarbeiter-Benutzer.

Muss es ein Administrator und/oder Moderator sein?

ETA: Wenn der Test mit einem Administrator wiederholt wird, um die direkte Antwort (mit Zitat + ErwÀhnung) zu erstellen, scheinen Àhnliche Ergebnisse erzielt zu werden.

Es gibt eine UX-Besonderheit, da jeder dieser Testbenutzer eine Benachrichtigung fĂŒr die jeweiligen Dinge erhalten hat und, als sie folgten, der Beitrag fĂŒr sie nicht mehr sichtbar war.

Es gibt auch eine weitere UX-Besonderheit fĂŒr die erweiterbare Box „x Antworten“, bei der die Anzahl das FlĂŒstern enthĂ€lt, es aber nicht anzeigt, wenn es erweitert wird.

1 „GefĂ€llt mir“

Unsicher, wir werden weitere Tests durchfĂŒhren und berichten, danke :smiley:

1 „GefĂ€llt mir“

Das ist jetzt nicht mehr der Fall :tada:

5 „GefĂ€llt mir“