Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Feature/481 wagtail newsletter #492

Draft
wants to merge 11 commits into
base: main
Choose a base branch
from

Conversation

mgax
Copy link
Member

@mgax mgax commented Jan 11, 2025

Fixes #481.

  • Install wagtail-newsletter and render some minimal MJML
  • Download newsletter archive from Mailchimp
  • Replicate the recent "This Week in Wagtail" issues listed in Port "This Week in Wagtail" to wagtail-newsletter #481 (comment)
  • Validate rendered newsletters using Litmus
  • Make sure the public view of a NewsletterPage looks good
  • Fix migrations and preserve the existing newsletter page content

wagtail-newsletter recommends installing the mrml library to render MJML, but the mrml package doesn't work on Python 3.13 (wagtail/wagtail-newsletter#69), so we use mjml-python (another wrapper of the same underlying Rust mrml library) for now.

@mgax mgax force-pushed the feature/481-wagtail-newsletter branch 3 times, most recently from 440c232 to c23960a Compare January 11, 2025 01:47
@mgax mgax force-pushed the feature/481-wagtail-newsletter branch from c23960a to 317c68c Compare February 25, 2025 18:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Port "This Week in Wagtail" to wagtail-newsletter
1 participant