Crochets pour la personnalisation des emails de résumé et de notification

Fonctionnalité
1

I have a client who would like to include ads in summary and notification emails. I’ve “crafted” a plugin that allows adding some text to the digest email (which I think needs some updating to catch up with changes made to that template).

I don’t like that I have to override the entire template to customize those emails.

Would it be possible to include a way to add to those emails without overriding the template?

The plugin works by having taking a topic ID in a custom field and including raw of the last post in the topic as the text added to the email (just before or after the header). Though I think it’s a pretty clever hack, that seems a bit too fiddly a solution to suggest adding to core. Perhaps there’s a way to have the Ad plugin be able to apply house ads to the digest and summary emails?

Managing a fragile plugin

Maybe I should write a spec that compares the digest email with the plugin off with the digest email with the plugin on but not configured to add anything to the email? Then I’d catch changes to the core template?

6 « J'aime »
2

Salut Jay

Ce plugin « fonctionne »-t-il avec Discourse 3.1 ? Ou vaut-il au moins la peine d’être testé ?

3

Vous pouvez essayer. Je ne pense pas avoir eu de leurs nouvelles depuis ce message, donc je suppose qu’ils n’ont pas mis à niveau depuis lors.

Vous pouvez me faire savoir si c’est cassé. Je n’aurai probablement pas le temps de le réparer gratuitement, mais je pourrais.

2 « J'aime »
4

D’après ce que je comprends, digest_custom_html et digest_custom_text implémentent déjà des hooks pour digest.html.erb et digest.text.erb. Donc, une implémentation de base de cette demande impliquerait d’ajouter plus de ces hooks ?

Une implémentation plus avancée pourrait fournir des sous-modèles, qui pourraient être utilisés pour fournir aux auteurs de plugins du code de style et de formatage maintenu par Discourse.

Est-ce que je comprends correctement que votre implémentation serait possible avec un digest_custom_html("before_preheader") ajouté à digest.html.erb ?

5

Je pense que oui. Je n’ai pas regardé de près comment cela fonctionne depuis des années.

6

Salut @thoka, as-tu réussi à implémenter quoi que ce soit via ces hooks ? Nous cherchons actuellement à injecter des segments sponsorisés simples via ces hooks.

7

Je l’ai fait. Tant que vous générez vos injections avec html_safe, tout va bien.

Il y a une discussion sur Fixing digest_custom_html to be treated as HTML (was: Overriding digest.html.erb)

2 « J'aime »