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

Update EmailComponent.php #168

Closed
wants to merge 2 commits into from
Closed

Update EmailComponent.php #168

wants to merge 2 commits into from

Conversation

zphelj
Copy link

@zphelj zphelj commented May 25, 2023

Added language to emails that direct people to not reply to the email, go to TALK instead.

Added language to emails that direct people to not reply to the email, go to TALK instead.
Comment on lines +87 to +89
Please do not reply to this email with questions regarding this event. For further assistance use our <a
href="https://talk.dallasmakerspace.org">Talk Forum</a>
<br><br>
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It probably makes more sense for this to be part of the footer section than here.
Further, if we do not want emails, we should probably make this sent from a noreply address as well.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I wanted the reference to Talk to be more aggressive than sticking it in the footer. I like the idea of a noreply address and have added that change.

@zphelj
Copy link
Author

zphelj commented Feb 1, 2025

This was incorporated in another PR. Closing.

@zphelj zphelj closed this Feb 1, 2025
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.

2 participants