Page MenuHomePhabricator

Fix two edge case issues with Differential mail tags
ClosedPublic

Authored by epriestley on Jan 9 2014, 1:53 AM.
Tags
None
Referenced Files
F17743972: D7912.id17909.diff
Jul 21 2025, 2:58 PM
F17738938: D7912.id17926.diff
Jul 21 2025, 1:16 AM
F17736636: D7912.diff
Jul 20 2025, 5:47 PM
Unknown Object (File)
Jun 18 2025, 7:52 AM
Unknown Object (File)
Jun 18 2025, 1:30 AM
Unknown Object (File)
Jun 6 2025, 12:17 AM
Unknown Object (File)
May 31 2025, 2:13 PM
Unknown Object (File)
May 30 2025, 10:07 AM
Subscribers

Details

Summary

Via Asana. The tags on Differential mail are wrong in two cases:

  • Transactions which submit inline comments but no comment text are not labeled as "comments", but should be.
  • Non-close, non-comment transactions are not labeled at all, but should be labeled "other".
Test Plan

Submitted a no-comments, inlines-only transaction and got a message with proper X-Phabricator-Mail-Tags header.

Diff Detail

Lint
Lint Skipped
Unit
Tests Skipped