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

Use sendmail_path as default #14433

Merged
merged 4 commits into from
Feb 22, 2024
Merged

Use sendmail_path as default #14433

merged 4 commits into from
Feb 22, 2024

Conversation

timkelty
Copy link
Contributor

@timkelty timkelty commented Feb 20, 2024

Description

This allows sendmail to work out of the box, without explicit settings, as configured by ini_get('sendmail_path').
This allows DDEV/Mailpit to work immediately.

  • Falls back to ini_get('sendmail_path') before resorting to \craft\mail\transportadapters\Sendmail::DEFAULT_COMMAND
  • Fixes selectize hint
  • Makes ini_get('sendmail_path') the first option in the dropdown

Related issues

@brandonkelly brandonkelly changed the base branch from develop to 4.8 February 21, 2024 13:15
@brandonkelly brandonkelly merged commit e01c36b into 4.8 Feb 22, 2024
@brandonkelly brandonkelly deleted the feature/use-sendmail_path branch February 22, 2024 00:16
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

2 participants