Add block support to bbPress emails about blocks-everywhere HOT 5 CLOSED

I've gone for a simple plain text version in #46 which replaces the content in the email with a nicer plain text version.

from blocks-everywhere.

The redesigned notification emails on WP.com support forums support block content. However, in bbPress core, there's a heavy HTML stripping with strip_tags. Basically, core bbPress emails are text only. We don't have that in the redesigned notifications emails. However, not sure what is the best course of action for bbPress core here. Push the fully redesigned notification emails to core? Probably, so that the blocks look good in the emails. But community might not like that. Or just remove the strip_tags call?

If we are going with the plugin approach for Gutenberg in bbPress core, I suppose we could add the redesigned emails there, too. However, we use a lot of internal WP.com HTML emails template building functionality so not sure how portable that is.

cc @lezama

from blocks-everywhere.

If we are going with the plugin approach for Gutenberg in bbPress core, I suppose we could add the redesigned emails there, too. However, we use a lot of internal WP.com HTML emails template building functionality so not sure how portable that is.

I think we should do this and add filters to separate WPCOM only logic.

from blocks-everywhere.

I would personally suggest taking the almost do-nothing approach for bbPress emails.

Ensure that the plain-text generated is good-enough, while not going above-and-beyond.

That could just mean stripping Block HTML out before bbPress strips it, so bbPress gets a "clean" text, and potentially doing some nice things like decoding code blocks and adding bullet points for lists.

from blocks-everywhere.

Closing this as done.

from blocks-everywhere.