Grupos adicionales no visibles en la página /about

Actualicé a Discourse 3.5.0.beta6-dev (da0b303568) esta mañana.

Después de la actualización, me encontré con un banner rojo en la parte superior del sitio que decía:

Aviso de administrador: estás usando el componente de tema discourse-add-groups-to-about. Esta función ahora está disponible en el núcleo de Discourse. Deberías eliminar este componente de tema.

Hice lo que se me indicó y eliminé el componente: Add groups to the /about page

Luego fui a la configuración de mi sitio de Discourse y volví a aplicar todas las configuraciones de la página “acerca de” que usé en el componente anterior:

Pero ninguno de los grupos aparece en la página “acerca de”:

Así es como solía verse, antes de que eliminara el componente:

También intenté verlo en Modo Seguro, pero no hubo diferencia.

1 me gusta

Creo que la configuración del sitio show_additional_about_groups todavía está oculta

Pero necesitas habilitar eso para que los grupos aparezcan.

6 Me gusta

Oh no :confused:

No puedo reinstalar el tema ahora porque el gran banner de advertencia rojo es visible para todos los usuarios, incluso para las personas que no han iniciado sesión :cry:

2 Me gusta

Si está autoalojado, puede habilitar la configuración

6 Me gusta

Gracias @Moin

¿Tengo que hacer esto manualmente o es probable que este error se corrija en una futura actualización? :thinking:

1 me gusta

Creo que una migración que se agregó hace 3 días debería haber habilitado esa configuración. DEV: Migrate add-groups-to-about component settings to site settings by Drenmi · Pull Request #32910 · discourse/discourse · GitHub Así que no tengo idea de si eso todavía sucederá en tu foro y por qué no sucedió cuando actualizaste :woman_shrugging:

5 Me gusta

Gran información, gracias :slight_smile:

¿La configuración show_additional_about_groups es visible en tu panel de administración? :thinking:

1 me gusta

No, pero no he instalado el componente. Quizás por eso no se produjo la migración para mí. Pero realmente no me interesaba porque no uso la función.

De hecho, sigo las PR en GitHub principalmente porque las capturas de pantalla ayudan con la traducción, especialmente para funciones como esta que no están disponibles en la interfaz al principio.

2 Me gusta

Pero si esto está ahora en el núcleo, ¿no está la opción disponible para todos ahora? :thinking:

1 me gusta

Está en el núcleo, pero la configuración show_additional_about_groups sigue siendo una configuración oculta del sitio, por lo que no es visible.
Creo que la migración la habilitó para los sitios que usaban el componente antes. No estoy seguro de si se habilitará para todos o simplemente se eliminará. Creo que solo es necesario durante el proceso de migración.

Si he entendido correctamente, la idea es que la configuración show_additional_about_groups se agregue al núcleo. Luego, se agrega una verificación al componente del tema para que los grupos solo se muestren si show_additional_about_groups es falso. Esto significa que simplemente continúa funcionando como antes. Después, todo se construye en el núcleo, pero solo se muestra si show_additional_about_groups es verdadero.
Después de que la configuración se haya migrado automáticamente, se activa show_additional_about_groups. Ahora el código del núcleo muestra los grupos y ya no el componente.
Esta migración se agregó hace 3 días. Luego, hace 2 días, se agregó el aviso en el componente que se te mostró.
No sé por qué la migración obviamente no funcionó para ti (ni la configuración se aplicó automáticamente, ni se activó show_additional_about_groups).

Creo que definitivamente es un descuido que la advertencia global no compruebe si el usuario es un administrador.

2 Me gusta

Gracias por la información @Moin, muy útil.

Hmm, ¿entonces supongo que está activado por defecto para todos, para todos los que no usaban previamente el componente de tema?

¿Y el mío simplemente no muestra ninguno de los grupos en la página “Acerca de” porque mi configuración sigue atascada en false en segundo plano?

No creerías cuántos correos electrónicos y consultas hemos recibido de nuestros miembros sobre este banner de advertencia :see_no_evil_monkey:

1 me gusta

@Moin Lo he aplicado a través de la consola de Rails:

SiteSetting.show_additional_about_groups = true

Y los grupos adicionales han vuelto a aparecer.

Gracias de nuevo por toda tu ayuda y tus ideas :person_bowing:

4 Me gusta

[cita=“Richie, post:3, topic:368218”]
el gran banner rojo de advertencia es visible para todos los usuarios
[/cita]

gracias por señalarlo, voy a arreglar esto para que otros no se topen con esto

4 Me gusta

¿Hay un error aquí aún por resolver o es solo un tema de #soporte en este momento?

1 me gusta

Pensé que la migración debería haber movido la configuración de @Richie del componente a la configuración central y también debería haber habilitado la configuración oculta del sitio show_additional_about_groups. Una razón por la que no funcionó podría ser que cambió el nombre, porque eso es fácilmente factible en la interfaz.
¿Hay alguna razón por la que la migración se basa únicamente en el nombre en lugar de también verificar un componente donde el remote_url de la tabla remote_themes coincida? Eso capturaría los componentes renombrados siempre y cuando se instalaran desde el repositorio oficial.

La configuración oculta del sitio, que no se habilitó porque la migración no ocurrió, impidió que el núcleo mostrara los grupos y, en cambio, el componente todavía mostraba los grupos. Pero el aviso global del componente le dijo que eliminara el componente. Entonces, el núcleo todavía no mostraba los grupos porque show_additional_about_groups todavía estaba deshabilitado, y no era fácilmente posible habilitarlo.

Entonces, si la migración automática no funcionó, ¿cómo migran los administradores manualmente? Copiar la configuración no es el problema. Pero, ¿cuándo cambian entre el componente que muestra los grupos y el núcleo que muestra los grupos sin necesidad de habilitar la configuración oculta?
Tal vez hubiera sido mejor habilitar show_additional_about_groups para todos o mostrarlo en la interfaz antes de agregar un aviso al componente de que debe eliminarse. Entonces, una migración manual habría funcionado, y el núcleo mostraría los grupos después, por lo que eliminar el componente no dejaría a los administradores sin ningún grupo en la página “acerca de”.

En este momento, cuando un administrador agrega grupos a la configuración del sitio about page extra groups, no sucede nada porque la configuración oculta show_additional_about_groups no está habilitada. Eso se siente como un error, aunque desde la perspectiva de un desarrollador, todo funciona como se espera. Creo que sería más fácil para un administrador entender lo que sucede si la configuración fuera visible en lugar de oculta.

4 Me gusta

@Moin ha resumido todo perfectamente.

Ninguna de mis configuraciones se copió y, cuando las copié, los grupos aún no eran visibles en la página /about porque la configuración oculta impedía que aparecieran y tuve que usar la consola de Rails para resolver el problema.

Así es como me quedé atascado:

Y esta es la eterna pregunta :blush:

3 Me gusta

@ted / @kris.kotlarek / @hugh ¿alguna opinión al respecto? ¿La migración que hizo Ted necesita un refinamiento?

Esto se siente muy extraño, ¿por qué no simplemente eliminar la configuración del sitio mostrar_grupos_adicionales_acerca_de y simplemente dejarla en verdadero?

Una configuración menos de la que preocuparse y la configuración de grupos adicionales de la página acerca de siempre funcionará como se espera.

4 Me gusta

Hola, @Richie, @Moin :waving_hand:

Hemos utilizado esta configuración oculta del sitio en el proceso de portar el componente de tema. Debido a un problema con la migración, no se habilitó correctamente cuando se migraron los datos.

Ahora que se ha completado la portabilidad, ya no necesitamos esta configuración, como señaló @sam. Simplemente podemos ver la lista de grupos configurados.

Hay una PR que hace eso aquí:

5 Me gusta

¡Genial! Pensé que tendría que quedarse hasta la próxima versión estable para asegurarme de que el componente no muestre los grupos además del núcleo para aquellos que migren entonces.

4 Me gusta

Gracias por la actualización @ted :slightly_smiling_face:

2 Me gusta