Restore Mailing List Mode Daily Summary

Настройка Плагин
42

It doesn’t disable the digest but it does provide another option.

Exactly! If you’re limiting what goes out in the summary, it now becomes a digest. Which is the default setup in Discourse.

2 лайка
43

hi @joebuhlig and @pfaffman,

thanks for your replies. but i don’t really get it and maybe you can help me out:

what settings would I need to change to change the current behaviour (ALL topics are included in the daily summary even if the reached the user already during the day because the watch the category)?

thanks in advance,
etienne

44

If I understand correctly, all you need to do is turn off the mailing-list-mode-daily-summary plugin.

The thiing is that the summary might not include ALL of the posts for the day, it chooses just the top 5 or something. You can see what the normal summary looks like at (something like) /admin -> email -> summary.

1 лайк
45

ah - now i get what you are telling me.

as we need ALL messages to get to our users in the daily summary using the build-in function is not an option. it does not send out all messages.

thats why we are using the mailing-list-mode-daily-summary plugin in the first place.

but now we are getting comments from users about getting messages twice: first as mail during the day because they are watching a topic and then later in the mlm-daily-summary again.

but probably it is not consistent with the idea of a daily SUMMARY to exclude certain messages (that have been send to the user already). so users have to get used to getting things twice i guess.

46

If your users watch the categories that they want they will get all of the messages. They do get each one individually rather than a single message with all of them.

People who watch a category or visit the site regularly don’t need mailing list mode or the plugin.

47

Sounds like you have a conflict between the staff’s desires and the users’ desires. Staff wants everyone to see everything, but the users only want to see a summary.

I’m guessing you’ll need to rectify that discrepancy first.

2 лайка
48

yes, you are right @joebuhlig. we’ll decide on that in the team.

as for your proposal of paying 200$ for the bugfixes: we are discussing that tomorrow in a team-meeting. will let you know.

2 лайка
49

hi @joebuhlig,

sorry - i forgot to tell you earlier: i couldnt bet through with my proposal of paying you guys for fixing the bug. so we would wait for you and your team to find time to fix it.

we are looking forward to seeing the bug fixed.

best, etienne

50

Привет, @joebuhlig и всем, кто использует этот плагин. У кого-нибудь ещё возникают проблемы с этим плагином после обновления до версии 2.3.0 Discourse? Когда наш хост обновил нас до версии 2.3.0 несколько недель назад, ежедневные письма перестали отправляться. Кроме того, когда я захожу в экран настроек электронной почты пользователя, флажок для этой опции не сохраняется. Можно нажать на него и сохранить, но после перезагрузки он снова снимается. Интересно, нашёл ли кто-нибудь способ исправить эти проблемы. Большое спасибо, если у кого-то есть какие-либо идеи!

1 лайк
51

Привет, Леа, проверь мой предыдущий пост от 29 января и узнай, не связана ли твоя проблема тоже с записью в user_custom_fields? С уважением, Этьен.

1 лайк
52

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

@etienne, спасибо за то, что поделились своими выводами насчёт true/false (t/f). Мы попросили кого-то разобраться в этом, и он сказал, что код, похоже, справляется с этим корректно. Поэтому в нашей ситуации я всё ещё не понимаю, почему некоторые пользователи не получают ежедневные письма, а другие получают их только раз в несколько дней. У нас точно появляются новые темы и новые посты каждый день, так что это письмо должно отправляться ежедневно.

Наш разработчик WordPress (который не является экспертом по Discourse или Ruby, а просто человек, готовый разобраться в проблеме) действительно удалось исправить ошибку JS на фронтенде, из-за которой чекбокс не сохранялся.

Ещё один вопрос ко всем, кто использует этот плагин: подозреваете ли вы, что плагин может вызывать сбои и блокировать таблицы проблемным образом? У нас есть ещё одна загадочная проблема с форумом, и одна из теорий заключается в том, что её вызывает этот плагин из-за заблокированных таблиц, которые не разблокируются, но мы пока не можем этого точно подтвердить.

53

Привет,

Я обновил свой Discourse до версии 2.4.0 beta2. С тех пор этот плагин не работает.

В данный момент мой код ошибки в Sidekick:

Jobs::HandledExceptionWrapper: Wrapped NoMethodError: undefined method apply_notification_styles’ for #UserNotifications:0x00007f1437382668`

Надеюсь, что плагин скоро исправят.

54

Мы тоже это видим.

55

Вероятно, это связано с тем, что мы унифицируем стилизацию всех писем, чтобы сделать их более тематическими, благодаря @neil.

56

Я только что выполнил пересборку, поэтому это актуально для последней версии v2.4.2.beta2 Discourse:

Вчера мы отключили плагин mlm-daily и очистили очередь повторных попыток. Однако ошибки всё ещё появляются в /logs, а повторные попытки продолжают накапливаться в Sidekiq:

Jobs::HandledExceptionWrapper: Wrapped NoMethodError: undefined method apply_notification_styles’ for #UserNotifications:0x00007f6f971b9168`

Похоже, это влияет на все уведомления, а не только на ежедневные сводки. Есть ли обходной путь, который мы можем реализовать до завершения полной переработки стилей электронной почты?

Спасибо,
Гуннар

57

Это связано с используемым вами сторонним плагином. Вам необходимо либо обновить этот плагин, либо отключить его.

58

Мы наблюдаем эту проблему и в нашей установке (2.4.0 beta4). Нужно ли мне что-то обновить, чтобы это исправить?

Кроме того, когда пользователи пытаются включить режим рассылки в своих настройках, это сохраняется (по крайней мере, интерфейс так показывает), но при повторном открытии диалога настроек флажок снова снимается. Таким образом, мы больше не можем заставить этот плагин отправлять ежедневные сводные письма. Я не уверен, связано ли это с сообщением Jobs::HandledExceptionWrapper, упомянутым выше, но, думаю, нет.

Что я могу сделать, чтобы это исправить?

Дирк

59

Кто-то должен обновить плагин, чтобы решить проблему. Вы можете прочитать темы howto для разработчиков плагинов или написать в Marketplace, если у вас есть бюджет.

3 лайка
60

Привет, @joebuhlig.

Вы всё ещё готовы поработать над этим плагином, обновить его для предстоящей версии Discourse 2.4 и исправить ошибку, описанную ранее, на платной основе? Если вы назовёте цену (ранее в этом году вы указывали 200 долларов за исправление ошибки), мы обсудим это в нашей команде.

Спасибо, что сообщили нам.
Этьен

61

Предисловие:

Это не совсем обновление плагина, а скорее горячее исправление. Оно не исправляет плагин для работы с новой системой, а лишь интегрирует удалённые части обратно в плагин. Это решение не является долгосрочным. И оно точно не исправит того, что команда Discourse сочла ежедневные сводки ненужным частным случаем, а не важной функцией для удержания пользователей, предпочитающих получать уведомления по электронной почте. Discourse больше нельзя использовать как замену рассылке, и если у вас есть возможность перейти на другую платформу — лучше это сделать. Также это не исправит позицию «нам не нужна документация или нормальные заметки о выпуске» < /rant>

Тем не менее. Мы используем это в продакшене, и всё работает. Это не идеально, но что поделать.

Как снова начать получать письма

  1. Примените исправление от @lkramer:
    Restore Mailing List Mode Daily Summary - #52 by lkramer

Теперь вы снова можете переключать кнопку в интерфейсе.

  1. Верните метод apply_notification_styles
    В файле engine.rb добавьте следующее в строку 11 (первый метод в классе):
 def apply_notification_styles(email)
                   email.html_part.body = Email::Styles.new(email.html_part.body.to_s).tap do |styles|
                   styles.format_basic
                   styles.format_html
                   end.to_html
                   email

Обратите внимание: исходная функция (см. коммит, который всё сломал, чтобы ознакомиться) использовала styles.format_notification вместо styles.format_html. Поскольку _notification был удалён, мы просто используем _html. Хорошая ли это идея? Нет. Работает ли это? Похоже, что да. По крайней мере, у нас есть небольшой шанс, что это не будет удалено при следующем обновлении.

  1. «Змеиное масло»

Я также обернул весь файл mailing_list.html.erb в тег Div с классом summary-email и отключил темизацию для сводных писем в настройках. Это было отчаянной попыткой, которая сначала не дала результатов. Скорее всего, вы обойдётесь и без этого, но я больше не буду трогать этот беспорядок, пока он снова не сломается. Поэтому, если у вас возникнут проблемы с форматированием, не стесняйтесь попробовать это.

Надеюсь, это поможет.

1 лайк