Zusätzliche Gruppen auf der Seite /about nicht sichtbar

Ich habe heute Morgen auf Discourse 3.5.0.beta6-dev (da0b303568) aktualisiert.

Nach dem Update wurde ich von einem roten Banner am oberen Rand der Website begrüßt, auf dem stand:

Admin-Hinweis: Sie verwenden die discourse-add-groups-to-about-Theme-Komponente. Diese Funktion ist jetzt in Discourse Core verfügbar. Sie sollten diese Theme-Komponente entfernen.

Ich habe getan, was mir gesagt wurde, und die Komponente gelöscht: Add groups to the /about page

Anschließend habe ich die Einstellungen meiner Discourse-Website aufgerufen und alle Einstellungen für die Über-uns-Seite, die ich in der alten Komponente verwendet habe, neu angewendet:

Aber keine der Gruppen wird auf der Über-uns-Seite angezeigt:

So sah es früher aus, bevor ich die Komponente gelöscht habe:

Ich habe auch versucht, im abgesicherten Modus anzuzeigen, aber es gab keinen Unterschied.

1 „Gefällt mir“

Ich glaube, die show_additional_about_groups-Site-Einstellung ist immer noch versteckt

Aber Sie müssen das aktivieren, damit die Gruppen angezeigt werden.

6 „Gefällt mir“

Oh nein :confused:

Ich kann das Theme jetzt nicht neu installieren, da das große rote Warnbanner für alle Benutzer sichtbar ist, sogar für nicht angemeldete Personen :cry:

2 „Gefällt mir“

Wenn Sie selbst gehostet sind, können Sie die Einstellung aktivieren

6 „Gefällt mir“

Danke @Moin

Muss ich das manuell machen oder wird dieser Fehler wahrscheinlich in einem zukünftigen Update behoben? :thinking:

1 „Gefällt mir“

Ich glaube, eine Migration, die vor 3 Tagen hinzugefügt wurde, hätte diese Einstellung aktivieren sollen. DEV: Migrate add-groups-to-about component settings to site settings by Drenmi · Pull Request #32910 · discourse/discourse · GitHub Also, keine Ahnung, ob das auf deinem Forum immer noch passiert und warum es nicht passiert ist, als du aktualisiert hast :woman_shrugging:

5 „Gefällt mir“

Tolle Einsicht, danke :slight_smile:

Ist die Einstellung show_additional_about_groups in Ihrem Admin-Panel sichtbar? :thinking:

1 „Gefällt mir“

Nein, aber ich habe die Komponente nicht installiert. Vielleicht ist das der Grund, warum die Migration bei mir nicht stattgefunden hat. Aber ich war nicht wirklich interessiert, weil ich die Funktion nicht nutze.

Tatsächlich verfolge ich die PRs auf GitHub hauptsächlich, weil die Screenshots bei der Übersetzung helfen, insbesondere bei Funktionen wie dieser, die zunächst nicht in der Benutzeroberfläche verfügbar sind.

2 „Gefällt mir“

Aber wenn das jetzt im Core ist, ist die Option dann nicht für jeden verfügbar? :thinking:

1 „Gefällt mir“

Es ist im Core, aber die Einstellung show_additional_about_groups ist immer noch eine versteckte Site-Einstellung, also nicht sichtbar.
Ich glaube, die Migration hat sie für Sites aktiviert, die die Komponente zuvor verwendet haben. Ich bin mir nicht sicher, ob sie für alle aktiviert oder einfach entfernt wird. Ich denke, sie wird nur während des Migrationsprozesses benötigt.

Wenn ich es richtig verstanden habe, ist die Idee, dass die Einstellung show_additional_about_groups zum Core hinzugefügt wird. Dann wird der Theme-Komponente eine Prüfung hinzugefügt, sodass die Gruppen nur angezeigt werden, wenn show_additional_about_groups falsch ist. Das bedeutet, dass es einfach wie zuvor weiter funktioniert. Danach ist alles im Core integriert, wird aber nur angezeigt, wenn show_additional_about_groups wahr ist.
Nachdem die Einstellungen automatisch migriert wurden, wird show_additional_about_groups aktiviert. Nun zeigt der Code aus dem Core die Gruppen an und nicht mehr die Komponente.
Diese Migration wurde vor 3 Tagen hinzugefügt. Dann wurde vor 2 Tagen der Hinweis in der Komponente hinzugefügt, der Ihnen angezeigt wurde.
Ich weiß nicht, warum die Migration offensichtlich nicht für Sie funktioniert hat (weder die Einstellungen wurden automatisch angewendet, noch wurde show_additional_about_groups aktiviert).

Ich denke, es ist definitiv ein Versehen, dass die globale Warnung nicht prüft, ob der Benutzer ein Administrator ist.

2 „Gefällt mir“

Danke für die Einblicke @Moin - sehr nützlich.

Hmm, ich schätze, das ist dann standardmäßig für alle aktiviert, für alle, die die Theme-Komponente nicht zuvor verwendet haben?

Und meine zeigt einfach keine der Gruppen auf der Über-uns-Seite an, weil meine Einstellung im Hintergrund immer noch auf false steht?

Du würdest nicht glauben, wie viele E-Mails und Anfragen wir von unseren Mitgliedern zu diesem Warnbanner erhalten haben :see_no_evil_monkey:

1 „Gefällt mir“

@Moin Ich habe dies über die Rails-Konsole angewendet:

SiteSetting.show_additional_about_groups = true

Und die zusätzlichen Gruppen sind jetzt wieder aufgetaucht.

Vielen Dank nochmals für all deine Hilfe und Einblicke :person_bowing:

4 „Gefällt mir“

[quote=„Richie, Beitrag:3, Thema:368218”]
Das große rote Warnbanner ist für alle Benutzer sichtbar
[/quote]

Danke, dass Sie darauf hingewiesen haben. Ich werde das beheben, damit andere nicht darauf stoßen.

4 „Gefällt mir“

Gibt es hier noch einen zu behebenden Fehler oder handelt es sich an diesem Punkt nur noch um ein #Support-Thema?

1 „Gefällt mir“

Ich dachte, die Migration hätte @Richies Konfiguration von der Komponente in die Haupteinstellungen verschieben und auch die versteckte Seiteneinstellung show_additional_about_groups aktivieren sollen. Ein Grund, warum es nicht funktioniert hat, könnte sein, dass er den Namen geändert hat, da dies in der Benutzeroberfläche leicht möglich ist.
Gibt es einen Grund, warum sich die Migration nur auf den Namen verlässt, anstatt auch nach einer Komponente zu suchen, bei der die remote_url aus der Tabelle remote_themes übereinstimmt? Das würde umbenannte Komponenten erfassen, solange sie aus dem offiziellen Repository installiert wurden.

Die versteckte Seiteneinstellung, die nicht aktiviert wurde, weil die Migration nicht stattgefunden hat, verhinderte, dass der Core die Gruppen anzeigt, und stattdessen zeigte die Komponente immer noch die Gruppen an. Aber der globale Hinweis der Komponente forderte ihn auf, die Komponente zu löschen. Dann zeigte der Core immer noch nicht die Gruppen an, weil show_additional_about_groups immer noch deaktiviert war, und es war nicht einfach möglich, sie zu aktivieren.

Wenn also die automatische Migration nicht funktioniert hat, wie migrieren Administratoren manuell? Das Kopieren der Konfiguration ist nicht das Problem. Aber wann wechseln sie zwischen der Anzeige der Gruppen durch die Komponente und der Anzeige der Gruppen durch den Core, ohne die versteckte Einstellung aktivieren zu müssen?
Vielleicht wäre es besser gewesen, show_additional_about_groups für alle zu aktivieren oder in der Benutzeroberfläche anzuzeigen, bevor ein Hinweis zur Komponente hinzugefügt wird, dass sie gelöscht werden soll. Dann hätte eine manuelle Migration funktioniert, und der Core würde die Gruppen anschließend anzeigen, sodass das Löschen der Komponente Administratoren nicht ohne Gruppen auf der Über-Seite zurücklassen würde.

Im Moment passiert nichts, wenn ein Administrator Gruppen zur Seiteneinstellung about page extra groups hinzufügt, weil die versteckte Einstellung show_additional_about_groups nicht aktiviert ist. Das fühlt sich wie ein Fehler an, auch wenn aus Entwicklersicht alles wie erwartet funktioniert. Ich denke, es wäre für einen Administrator einfacher zu verstehen, was passiert, wenn die Einstellung sichtbar statt versteckt wäre.

4 „Gefällt mir“

@Moin hat alles perfekt zusammengefasst.

Keine meiner Einstellungen wurden übernommen, und als ich sie kopierte, waren die Gruppen auf der /about-Seite immer noch nicht sichtbar, da die versteckte Einstellung verhinderte, dass sie angezeigt wurden, und ich die Rails-Konsole verwenden musste, um das Problem zu lösen.

So bin ich stecken geblieben:

Und das ist die ewige Frage :blush:

3 „Gefällt mir“

@ted / @kris.kotlarek / @hugh irgendwelche Gedanken dazu? Muss die von Ted durchgeführte Migration verfeinert werden?

[Zitat=“Moin, Beitrag:17, Thema:368218”]
Im Moment passiert nichts, wenn ein Admin der Site-Einstellung about page extra groups Gruppen hinzufügt, da die versteckte Einstellung show_additional_about_groups nicht aktiviert ist. Das fühlt sich wie ein Fehler an, auch wenn aus Entwicklersicht alles wie erwartet funktioniert. Ich denke, es wäre für einen Admin einfacher zu verstehen, was passiert, wenn die Einstellung sichtbar statt versteckt wäre.

[/Zitat]

Das fühlt sich sehr komisch an, warum nicht einfach die Site-Einstellung show_additional_about_groups löschen und sie einfach auf “true” setzen?

Eine Einstellung weniger, um die man sich kümmern muss, und die Einstellung für zusätzliche Gruppen auf der “Über uns”-Seite funktioniert immer wie erwartet.

4 „Gefällt mir“

Hallo, @Richie, @Moin! :waving_hand:

Wir haben diese versteckte Site-Einstellung beim Portieren der Theme-Komponente verwendet. Aufgrund eines Problems bei der Migration wurde sie bei der Datenmigration nicht korrekt aktiviert.

Da der Port jetzt abgeschlossen ist, benötigen wir diese Einstellung nicht mehr, wie @sam bemerkt hat. Wir können uns einfach die Liste der konfigurierten Gruppen ansehen.

Hier gibt es einen PR, der das macht:

5 „Gefällt mir“

Großartig! Ich dachte, es müsste bis zur nächsten stabilen Version bleiben, um sicherzustellen, dass die Komponente neben Core nicht auch die Gruppen für diejenigen anzeigt, die dann migrieren.

4 „Gefällt mir“

Danke für das Update @ted :slightly_smiling_face:

2 „Gefällt mir“