Удаление ember-addons/ember-computed-decorators

@markvanlan Перемещение декораторов из ember-addons/ember-computed-decorators в discourse-common/utils/decorators вот-вот сломает множество плагинов и тем, как только пользователи нажмут большую синюю кнопку обновления.

Возможно ли оставить существующие декораторы на месте (в дополнение к новым), чтобы дать нам время на переход?

Это не должно стать изменяющим изменением. Здесь, в файле discourse-loader, мы добавили алиас, чтобы при любой попытке импорта ember-addons/ember-computed-decorators автоматически загружался discourse-common/utils/decorator.

Да, я это видел. Не совсем понимаю, почему, но это, похоже, работает не во всех случаях. У меня возникает много ошибок RequireJS, которые исчезают, как только я перехожу на новый импорт.

Можете проверить, происходит ли это при использовании require вместо import?

Это была ошибка, которую, как я думал, я уже исправил.

Привет,

Вот сообщение об ошибке, с которым я столкнулся на своём сайте Discourse:

Screen Shot 2020-02-19 at 2.50.56 PM

Нужно ли это всё ещё обновлять в коде?

Я не могу точно определить, откуда возникает эта ошибка (какой плагин использует ember-addons/ember-compted-decorators), но в конечном итоге её всё же нужно исправить.

Это предупреждение, а не ошибка, потому что ember-addons/ember-compted-decorators пока ещё работает.

Спасибо @markvanlan,

Какой метод вы используете или рекомендуете для определения плагина, который использует устаревшие ресурсы?

Посмотрите, какие плагины установлены на вашем экземпляре, затем перейдите в каждый репозиторий на GitHub и выполните поиск по запросу ember-addons/ember-computed-decorators.

Полагаю, что некоторым из наших официальных плагинов Discourse всё ещё требуется это обновление, поэтому он может находиться в одном из наших репозиториев.

Можете ли вы развернуть это и увидеть полный стек вызовов?

Я не знаю, как увидеть трассировку стека за предупреждением в консоли браузера?

Однако я умею искать в плагинах через grep :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

Привет, сейчас я не могу воспроизвести эту проблему. Но феноменальный @RGJ только что обновил плагины на нашем сайте, и, я думаю, это решило ошибку? Он указал мне на эти обновления:

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

Отличный grep, Ричард! :slight_smile:

Я не думаю, что стоит сильно паниковать.

Снижение количества этих предупреждений — также ответственность разработчиков плагинов, и делать это нужно как можно скорее.

Недавно я уже устранил довольно много таких предупреждений, и впереди ещё будет немало. Это бесконечная история :wink:

Это просто предупреждения, и беспокоиться о них не стоит, если только их не игнорировать слишком долго.

Спасибо. Мне нравится, что Discourse продолжает совершенствоваться и поддерживать свой код!