¡Los GIFs se integran en el núcleo de Discourse!

No lo creo. Este caso parece similar a las cuadrículas de imágenes. En ese cambio también se mostró la advertencia, pero esta fue la respuesta que obtuve en ese momento:

1 me gusta

Todavía deberías poder desactivarlo a través de la configuración normal del sitio después.

@davidb ¿puedes confirmarlo?

Estoy de acuerdo en que esta redacción en los próximos cambios es algo que también deberíamos encontrar una manera de hacer menos confuso.

Una cosa que podemos intentar hacer es dejarlo claro en el título y la descripción del cambio en sí, cuando sea configurable.

1 me gusta

El próximo cambio de GIFs está actualmente bajo una configuración oculta del sitio, pero creo que en el futuro querramos que sea visible y que se pueda habilitar/deshabilitar dentro del área de administración.

Por ahora, simplemente mantén el próximo cambio como ‘Ninguno’ y estará desactivado por defecto en el futuro. No es necesario hacer ningún otro cambio en el lado de la administración.

4 Me gusta

Sí, cuando busqué enable_gifs me llevó directamente a la página de cambios futuros. Fue muy confuso.

El concepto de los cambios futuros es genial. Solo necesitaría un ajuste de lenguaje.

¡Gracias a todos!

2 Me gusta

En este caso, @davidb @mcwumbly, si el tipo de cambio hubiera sido site_setting_default en lugar de feature para enable_gifs, no habría aparecido el mensaje permanente alarmante. Esto se controla aquí:

Sin embargo, creo que en general el tipo feature está bien para este cambio próximo… así que quizás necesitemos hacer que este aviso permanente sea algo más explícito para los cambios próximos.

¿Quizás will_become_permanent: true o algo así como metadatos adicionales? No estoy seguro…

1 me gusta

¿por qué nos obligaron a usar Klipy en lugar de otras opciones? Prefiero Giphy y me gustaría seguir usándolo.

La configuración y la interacción con el componente hacen que sea muy confuso usar otras opciones como Giphy.

Creo que se debe a que Giphy ya no es una opción gratuita

1 me gusta

Ya está bien. Lo averigüé (¡gracias, Moin!). Tuve que desinstalar el componente de GIF de Discourse para que apareciera la configuración. Luego, establecí la configuración en ninguno, y después reinstalé y reconfiguré el componente. Giphy vuelve a funcionar para mí.

2 Me gusta

Me alegra haber podido ayudarte. El hecho de que el cambio próximo quede oculto cuando está instalado el componente de tema GIF hace que esto sea un poco confuso: para usar el componente, necesitas desinstalarlo, desactivar el cambio próximo y luego volver a instalarlo.
¿Te notificaron sobre el cambio o se suprimieron esas notificaciones porque el cambio próximo estaba oculto? Me pregunto si eso provoca el problema de que no sepan que deben ejecutar la migración y obtener una clave API.

1 me gusta

Al solicitar una clave KLIPY, piden que se muestre el atribución. ¿Cómo podemos añadir eso al popup? Y ¿cómo la están usando las personas sin ello?

1 me gusta

Pues solíamos tener esto en el componente del tema, pero no lo tenemos en el núcleo; había un providerLogo:

El de Klipy se añadió en FEATURE: Add Klipy as GIF API provider by xfalcox · Pull Request #107 · discourse/discourse-gifs · GitHub . @davidb, ¿fue intencional dejar este logotipo de proveedor fuera de la versión principal?

2 Me gusta

Lamentablemente, parece que se omitió al migrarlo al núcleo. Antes se mostraba de forma condicional en el componente del tema, pero ahora deberíamos mostrarlo siempre.

Se ha añadido aquí: UX: add Klipy branding to gifs picker - Pull Request #41501 - discourse/discourse - GitHub

2 Me gusta

Recientemente se añadió un banner de aviso global al componente del tema, de modo que cuando el componente esté actualizado, mostrará el banner y un enlace a este tema para los administradores del sitio.

Dado que la semana pasada movimos el cambio próximo a la rama estable, ahora debería aparecer ese banner en cualquier sitio que estuviera usando el componente del tema, para que puedan tomar las acciones necesarias.

Solo un aviso: podría valer la pena hacer un fork del repositorio si realmente quieres seguir usando giphy, ya que planeamos archivarlo en un futuro cercano.

2 Me gusta

No estoy seguro de que los requisitos de atribución se hayan leído siquiera. La marca es solo 1 atribución y es opcional:


https://docs.klipy.com/attribution

Tenemos nuestro propio servidor y utilizamos Giphy en el componente del tema antiguo. Desactivamos esa opción y ahora solo vemos las opciones de la API de Klipy. ¿Hay alguna forma de introducir nuestra antigua clave de Giphy y utilizarla como proveedor, o debemos usar Klipy ahora?

EDIT: Parece que no, así que estamos a merced del video de verificación a través de Klipy.

hola @fearlessfrog :wave:
tuve el mismo problema. Para seguir usando Giphy, debes eliminar completamente el componente del tema (desactivarlo no es suficiente), luego configurar la opción admin -> cambios futuroshabilitar gifs en “nadie”. Después, vuelve a instalar un fork del componente para poder usarlo, ya que el equipo archivará su versión pronto. Tengo un fork del componente aquí que puedes usar.

desinstalar el componente antiguo:

desactivar la configuración en admin-cambios futuros

realmente no entiendo por qué el equipo lo hizo de esta manera, pero es terriblemente confuso y me llevó un tiempo entenderlo (gracias a la explicación de Moin). Es muy poco amigable para los autoalojados. En algún momento probablemente reconstruiré el fork como mi propio componente, sin los otros campos y archivos innecesarios, pero publicaré un mensaje en el banner de administración cuando eso ocurra.

nota para el equipo: el componente Discourse Gifs aún aparece en la página de instalación de componentes predeterminada:

4 Me gusta

Gracias, @davidb, ¿podrías eliminar esta opción, por favor?

Lo hicimos de esta manera porque optimiza la configuración de nuestros clientes alojados, en cuyo caso entramos manualmente y desinstalamos el componente antiguo de gifs por ellos. No anticipamos que la gente quisiera seguir usando Giphy, dado que anunciaron precios tan altos, razón por la cual añadimos Klipy al componente de tema en primer lugar.

La próxima vez podemos añadir un poco más de información al mensaje original para quienes autoalojan y desean seguir usando el componente antiguo; simplemente asumimos aquí que quienes autoalojan también migrarían a Klipy:

Definitivamente pensaremos en una manera de hacer esto menos confuso y sorprendente; le he pedido a David B que abra un tema interno y trabajaré en ello en un futuro cercano.

2 Me gusta

Lo que no entiendo es que sigo usando mi vieja API gratuita de Giphy sin problemas. Acabo de iniciar sesión en el panel de desarrolladores de Giphy y ahí está. ¿Cuáles son estos cargos que supuestamente debería estar incurriendo? Nunca he tenido que ingresar una tarjeta de crédito ni pagar nada, y aún no veo dónde tengo que hacerlo ahora; para mí, siempre ha sido así. El foro al que lo tengo vinculado no parece tener límite de tasa, y tendemos a usarlo mucho :thinking:

Vale, parece que funciona perfectamente para hasta 100 llamadas a la API por hora, lo cual me parece perfectamente aceptable para los autoalojados.

2 Me gusta

Así es, sin duda. Sin embargo, el coste sería prohibitivo en nuestro alojamiento, considerando que queremos que los GIFs sean una función principal.

2 Me gusta

Creo que el componente del tema solo tenía «Powered by Klipy» (de esta lista), pero probablemente deberíamos haber sido más cuidadosos al verificarlo contra el sitio web oficial al portarlo al núcleo: hemos añadido el principal requerido en UX: add placeholder text for Klipy gifs search - Pull Request #41651 - discourse/discourse - GitHub

Hecho: añadido en UX: remove gifs from popular theme components - Pull Request #41649 - discourse/discourse - GitHub

Hemos actualizado la lógica sobre cómo aparece esta advertencia. Debería estar resuelto ahora en DEV: add upcoming change permanent banner metadata - Pull Request #41700 - discourse/discourse - GitHub

Estos problemas deberían estar resueltos ahora para los sitios que ejecutan la última versión de Discourse.

4 Me gusta