Page MenuHomePhabricator

Restore "Branch" and "changes since last update" fields to Differential mail
ClosedPublic

Authored by epriestley on Mar 31 2014, 7:28 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Mar 20, 12:49 AM
Unknown Object (File)
Wed, Mar 20, 12:49 AM
Unknown Object (File)
Wed, Mar 20, 12:49 AM
Unknown Object (File)
Wed, Mar 20, 12:03 AM
Unknown Object (File)
Tue, Mar 19, 11:40 PM
Unknown Object (File)
Feb 20 2024, 9:00 PM
Unknown Object (File)
Feb 4 2024, 3:14 AM
Unknown Object (File)
Feb 1 2024, 8:54 PM

Details

Summary

Ref T418. Fixes T4642. The "changes since last update" and "branch" fields got dropped; restore them in a general, field-driven way.

Test Plan
  • Created a revision, got relevant sections in mail.
  • Commented on a revision, got relevant sections in mail.
  • Updated a revision, got relevant sections in mail.

Diff Detail

Repository
rP Phabricator
Branch
dxmail
Lint
Lint Passed
Unit
Tests Passed

Event Timeline

epriestley retitled this revision from to Restore "Brach" and "changes since last update" fields to Differential mail.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: btrahan.
chad retitled this revision from Restore "Brach" and "changes since last update" fields to Differential mail to Restore "Branch" and "changes since last update" fields to Differential mail.Mar 31 2014, 8:16 PM

Thanks for this!

src/applications/differential/customfield/DifferentialBranchField.php
62

It's also useful to have the branch shown when a diff is accepted and ready to land.

(Out of curiosity, do you use arc branch? And/or would arc land D123 be useful? I assume you're using the BRANCH field to copy-paste to the CLI?)

  • Also show the field for accepted revisions.

Ooh! No, I didn't know about arc branch. I do think being able to say arc land D123 would also be useful.

btrahan edited edge metadata.
This revision is now accepted and ready to land.Mar 31 2014, 9:35 PM
epriestley updated this revision to Diff 20543.

Closed by commit rPcf6f7446ce3f (authored by @epriestley).