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.

Sign up for GitHub

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

Jump to bottom

Mail Settings #119

Merged
merged 3 commits into from
Dec 8, 2023
Merged

Mail Settings #119

merged 3 commits into from
Dec 8, 2023

Conversation

annlickander
Copy link
Contributor

  • Fix bug where we don't check if the site in a multisite should use SMTP or Postal to send emails
  • Add tab to site settings for mail settings
  • Make it possible to change the username in an email for sending emails. Default to 'notifications'
  • Make it possible to change sender name
  • Make it possible to change the from email address
  • Make it possible for a site in a multisite to overrride from email address
  • Make it possible for a site in a multisite to override sender name

@annlickander
add a mail setting tab to admin, to let sites in a multisite override…
f496bea 
… the general settings in config. Also fix bug with site check not checking
@annlickander annlickander added the enhancement New feature or request label Dec 8, 2023
@annlickander annlickander self-assigned this Dec 8, 2023
@annlickander annlickander merged commit b98f6d6 into main Dec 8, 2023
1 check passed
@annlickander annlickander deleted the feature/eq-154-byt-avsandarmail-pa-notiser-fran-wordpress branch December 8, 2023 15:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ErikBernskiold ErikBernskiold ErikBernskiold approved these changes

Assignees

@annlickander annlickander

Labels
enhancement New feature or request
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@annlickander @ErikBernskiold