Eliminación de ember-addons/ember-computed-decorators

@markvanlan The move of the decorators from ember-addons/ember-computed-decorators to discourse-common/utils/decorators is about to break a number of plugins and themes as soon as folks hit the big blue update button.

Would it be possible to leave the existing decorators in place (in addition to the new ones) to give us time to make the transition?

3 Me gusta

This should not be a breaking change. Here in the discourse-loader file, we have added an alias, so that anytime you try and import ember-addons/ember-computed-decorators, it will automatically load discourse-common/utils/decorator instead.

3 Me gusta

Yeah, I saw that. I’m not sure why, but that doesn’t seem to be working in all cases. I’m gettting a lot of requirejs errors that seem to resolve once I move to the new import.

1 me gusta

Are you able to see if it is happening when using require instead of import?

That was a bug that I thought I had fixed.

4 Me gusta

Hola,

Este es el mensaje de error que encontré en mi sitio de Discourse:

Screen Shot 2020-02-19 at 2.50.56 PM

¿Todavía es necesario actualizar esto en la base de código?

No puedo ver exactamente de dónde proviene ese error (qué plugin está usando ember-addons/ember-compted-decorators), pero eventualmente debe ser corregido.

Eso es una advertencia, y no un error, porque ember-addons/ember-compted-decorators aún funciona por ahora.

5 Me gusta

Gracias @markvanlan,

¿Qué método utilizas o recomiendas para identificar un plugin que esté utilizando activos obsoletos?

Vería qué complementos tiene instalados en su instancia y luego iría a cada repositorio en GitHub y buscaría ember-addons/ember-computed-decorators.

Creo que algunos de nuestros complementos oficiales de Discourse aún necesitan esta actualización, por lo que podría ser uno de nuestros repositorios.

4 Me gusta

¿Puedes ampliarlo y ver el rastreo completo?

3 Me gusta

No sé cómo ver el backtrace detrás de una advertencia en la consola del navegador.

Sin embargo, sé cómo buscar en los plugins :smile:

$ grep -r ember-addons/ember-computed-decorators plugins/|grep import|awk -F / '{print $2}'|uniq|sort
discourse-canned-replies
discourse-local-dates
discourse-presence
discourse-push-notifications
discourse-saved-searches
procourse-memberships
6 Me gusta

Hola, actualmente no puedo reproducirlo. Pero el fenomenal @RGJ acaba de actualizar los plugins en nuestro sitio y creo que esto resolvió el error. Me indicó estas actualizaciones:

DEV: Find/replace deprecated decorators · discourse/discourse-data-explorer@277346b · GitHub

2 Me gusta

¡Buen grep, Richard! :slight_smile:

No creo que haya mucho de qué preocuparse.

También es responsabilidad de los creadores de plugins minimizar estas advertencias lo antes posible.

He resuelto varias recientemente y habrá más. Es una historia sin fin :wink:

Son solo advertencias y no hay de qué preocuparse realmente a menos que se ignoren durante demasiado tiempo.

2 Me gusta

Gracias. ¡Me encanta que Discourse siga mejorando y manteniendo su código!

4 Me gusta