Skip to content

fix(notifications): set default SMTP encryption value to prevent false validation error#9543

Merged
peaklabs-dev merged 1 commit intocoollabsio:nextfrom
ShadowArcanist:jean/smtp-encryption-default-value
Apr 29, 2026
Merged

fix(notifications): set default SMTP encryption value to prevent false validation error#9543
peaklabs-dev merged 1 commit intocoollabsio:nextfrom
ShadowArcanist:jean/smtp-encryption-default-value

Conversation

@ShadowArcanist
Copy link
Copy Markdown
Member

Changes

  • Set default value for SMTP encryption to be starttls instead of null

Issues

Notes:

This issue only happens on team level notification and UI shows StartTLS as selected by default but it is set to null by default so if user tries to save the changes it will throw encryption field is required (unless they manually select encryption option from the dropdown)

Category

  • Bug fix

Preview

Before:

image

After:

image

AI Assistance

  • AI was used (please describe below)

If AI was used:

  • Tools used: Jean + Claude (Opus 4.6)
  • How extensively: every change on this PR (except the PR description)

Contributor Agreement

Important

  • I have read and understood the contributor guidelines. If I have failed to follow any guideline, I understand that this PR may be closed without review.
  • I have searched existing issues and pull requests (including closed ones) to ensure this isn't a duplicate.
  • I have tested all the changes thoroughly with a local development instance of Coolify and I am confident that they will work as expected when a maintainer tests them.

Copy link
Copy Markdown
Member

@peaklabs-dev peaklabs-dev left a comment

Choose a reason for hiding this comment

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

Thanks for the PR.

@peaklabs-dev peaklabs-dev merged commit 13a1f86 into coollabsio:next Apr 29, 2026
2 checks passed
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