Категория email в требует reply_by_email_enabled?

supermathie · 20.Январь.2026 21:29:45

Мое предложенное исправление:

github.com/discourse/discourse

FIX: email to a category should not require enabling reply-by-email (#37227)

main ← email_in_without_reply_by_email

merged 06:01PM - 03 Feb 26 UTC

Supermathie

+2007 -1921

DEV: email/receiver tests should explicitly test alternative_reply_by_email_addr…esses Some of the email/receiver tests were using an alternative reply address in their data, implicitly testing that functionality. This commit modifies the test email fixtures to always use the main reply address, and added a separate test explicitly testing the functionality of `alternative_reply_by_email_addresses`. This serves the mission of configuring as little as possible to not rely on side effects. DEV: email/processor tests should explicitly configure reply-by-email Tests that rely on reply-by-email being configured should explicitly do so. They're now moved to a common context in the same style as email/receiver specs. FIX: email to a category should not require enable reply-by-email This commit removes the side effect of disabling category email address lookups when `reply_by_email_enabled` is not configured, restoring functionality to the situation where a category's `email_in` is configured without `reply_by_email_enabled`. Added tests for each of these situations. DEV: separate tests for email receiver by whether reply_by_email should be configured The impetus for this change is commit e05ef50c introducing an unintended side effect that *should* have caused tests to fail; specifically the situation where `email_in` is enabled and a category has an incoming email address configured, but `reply_by_email_enabled` is *not* enabled. The change in the referenced commit broke this configuration, but this was not noticed as the spec always configured `reply_by_email_enabled`. The only change made in this commit is reorganising the tests such that reply-by-email is only configured when necessary. Thus, this commit *alone* will break tests.

Тема		Ответов	Просм.
Reply To email address of post notification same as incoming category address? Support	3	672	30.07.2023
Private sub-category email-in Discourse::InvalidAccess Support email	11	477	17.02.2024
"Email in allowed groups" setting is too permissive Support email , mailing-list	4	405	30.01.2024
Configuring Incoming Email to Create Category Topics or Group Messages Support email-in	9	226	02.10.2025
Category not accepting "anonymous email" from known users Bug	28	3759	14.05.2026

Категория email в требует reply_by_email_enabled?

Связанные темы