Page MenuHomePhabricator

Render prose diffs in email as summaries
ClosedPublic

Authored by epriestley on Jun 10 2016, 3:30 PM.
Tags
None
Referenced Files
F14050569: D16098.id38729.diff
Thu, Nov 14, 7:22 PM
F14038925: D16098.id38729.diff
Mon, Nov 11, 2:51 AM
F14029008: D16098.id38729.diff
Fri, Nov 8, 6:43 PM
F14013534: D16098.id38729.diff
Sat, Nov 2, 7:35 AM
F13992206: D16098.id38729.diff
Tue, Oct 22, 3:09 PM
F13967218: D16098.diff
Oct 16 2024, 12:19 PM
F13966539: D16098.id38729.diff
Oct 16 2024, 8:26 AM
F13962652: D16098.id.diff
Oct 15 2024, 11:27 AM
Subscribers
None

Details

Summary

Ref T7643. When a large block of prose text is edited (like a wiki page), summarize the diff when sending mail.

For now, I'm still showing the whole thing in the web UI, since it's a bit more manageable there.

Also try to fix newlines in Airmail.

Test Plan

This web diff:

Screen Shot 2016-06-10 at 8.20.08 AM.png (546×621 px, 99 KB)

..became this mail diff:

Screen Shot 2016-06-10 at 8.18.29 AM.png (717×914 px, 116 KB)

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Render prose diffs in email as summaries.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Jun 10 2016, 4:28 PM
This revision was automatically updated to reflect the committed changes.