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

feat: support configuring a welcome email on lists. #1818

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

chirino
Copy link

@chirino chirino commented Apr 6, 2024

This welcome template will be used to send a welcome email to new list members once the subscription is confirmed (if confirmation is required).

This welcome template will be used to send a welcome email to new list members once the subscription is confirmed (if confirmation is required).

Signed-off-by: Hiram Chirino <hiram@hiramchirino.com>
@knadh
Copy link
Owner

knadh commented Apr 7, 2024

Thank you for the PR @chirino. This feature request has been pending for a long time: #206

However, the feature shouldn't be specific to welcome responses, but should be based on events.

  • Subscribe
  • Opt-in
  • Unsubscribe
  • Blocklist

It should be configurable from a new Lists -> Autoresponse menu.

If you're able to amend the PR, that's great. Otherwise, I'll hold off on this for now and try to incorporate this when I find time. Thanks again.

@chirino
Copy link
Author

chirino commented Apr 7, 2024

In terms of db modeling what were you thinking?

@joerosenthal
Copy link

@knadh, @chirino, any news on getting this PR merged in? Been using a very hacky workaround for my listmonk instance to add welcome messages, and I'd love to upgrade to a kosher version.

@knadh
Copy link
Owner

knadh commented May 14, 2024

Hi @joerosenthal. Unfortunately, this PR/idea will have to wait for a bit. I've been working on adding multi-user support (https://github.com/knadh/listmonk/tree/multiuser) which is a massive, critical change for v4.0.0. Can only consider bigger features that require planning once multi-user is merged.

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.

None yet

3 participants