Page MenuHomePhabricator

Special-case text sections with empty headers in emails.
AbandonedPublic

Authored by klimek on Jul 2 2015, 12:21 PM.
Tags
None
Referenced Files
F18715179: D13526.id.diff
Mon, Sep 29, 9:22 AM
F18705640: D13526.diff
Sun, Sep 28, 12:20 PM
F18653210: D13526.diff
Sun, Sep 21, 11:37 AM
F18435081: D13526.diff
Aug 31 2025, 9:49 AM
F18419537: D13526.diff
Aug 30 2025, 2:35 PM
F18413407: D13526.diff
Aug 30 2025, 7:35 AM
F18412441: D13526.id.diff
Aug 30 2025, 6:39 AM
F18048653: D13526.id.diff
Aug 3 2025, 1:41 PM
Subscribers

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

This allows admins to configure a translation that translates some
section headers like REVISION_SUMMARY to an empty string and have the section
blend into the email naturally.

Putting the patch out there so it's clearer what the size of the change is,
and a decision whether to accept this can be made easier.

Test Plan

Configured empty sections, sent email.

Diff Detail

Repository
rP Phabricator
Branch
allow-empty-header-sections
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7114
Build 7280: [Placeholder Plan] Wait for 30 Seconds
Build 7279: arc lint + arc unit

Event Timeline

klimek retitled this revision from to Special-case text sections with empty headers in emails..
klimek updated this object.
klimek edited the test plan for this revision. (Show Details)
klimek added a reviewer: epriestley.