wagtail / wagtail-newsletter

Send email newsletters based on Wagtail content
https://wagtail-newsletter.readthedocs.io
BSD 3-Clause "New" or "Revised" License
9 stars 3 forks source link

Preview campaign #33

Closed mgax closed 3 weeks ago

mgax commented 3 weeks ago

Fixes https://github.com/wagtail/wagtail-newsletter/issues/31

After some back and forth, we've changed how campaign data is saved: we require the user to save a revision, and base the campaign off of that revision's content. This way it's unambiguous what the campaign content will be; there's no room for the user to forget to save their changes on the Wagtail Page and/or have another user overwrite their work before the campaign is saved.

Therefore, I've changed the behaviour of the newsletter_recipients and newsletter_subject fields: they are no longer specially persisted regardless of revisions; they are now part of the regular revision content. The newsletter_campaign field is still persisted: it's not editable by the user, and it's not helpful to have it overwritten when restoring a previous revision.

mgax commented 3 weeks ago

The preview page's header is pretty basic at the moment. I wonder what it would take to make it more snazzy, maybe like the history page's header.

Screenshot 2024-06-05 at 15 35 30
Screenshot 2024-06-05 at 15 35 47