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

Support email subscriptions #478

Merged
merged 21 commits into from Oct 3, 2023
Merged

Support email subscriptions #478

merged 21 commits into from Oct 3, 2023

Conversation

thebaer
Copy link
Member

@thebaer thebaer commented Jun 25, 2021

This adds email subscription functionality, currently only with Mailgun support for now.

It includes database and style changes, so run:

writefreely db migrate
make ui

When finished, this closes T856.

TODO:

  • Support SQLite: migration
  • Support SQLite: email subscriber queries
  • Support SQLite: publish job queries
  • Fully support private instances

  • ☑ I have signed the CLA

This adds beginning email subscription functionality, with only MySQL support,
Mailgun support, and incomplete support for private instances. It includes
database changes, so run:

    writefreely db migrate

to use this feature.

Ref T856
@thebaer thebaer added this to the 1.0 milestone Jun 25, 2021
@thebaer thebaer modified the milestones: 0.14, 0.15 Sep 21, 2023
@thebaer thebaer marked this pull request as ready for review September 25, 2023 19:32
@thebaer
Copy link
Member Author

thebaer commented Sep 25, 2023

This is ready for testing / review now!

@thebaer thebaer mentioned this pull request Sep 25, 2023
@thebaer
Copy link
Member Author

thebaer commented Sep 25, 2023

With #777, realizing that some admins might want that password-resetting functionality but not want to enable newsletters. So should add a config option to enable or disable it for the instance, and include the checkbox in the config web UI.

@thebaer
Copy link
Member Author

thebaer commented Oct 3, 2023

Going to merge this now so we can get all following changes merged, as well. Remaining work is noted in T856.

@thebaer thebaer merged commit 64dcb56 into develop Oct 3, 2023
@thebaer thebaer deleted the letters branch October 3, 2023 14:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Development

Successfully merging this pull request may close these issues.

None yet

1 participant