Page MenuHomePhabricator

When creating a Phriction document, mark initial transactions as "create" transactions to fix weird email
ClosedPublic

Authored by epriestley on May 22 2019, 10:15 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Jun 5, 4:25 PM
Unknown Object (File)
May 25 2025, 9:22 PM
Unknown Object (File)
May 20 2025, 10:03 PM
Unknown Object (File)
May 8 2025, 10:55 PM
Unknown Object (File)
Apr 21 2025, 1:56 PM
Unknown Object (File)
Apr 5 2025, 3:07 AM
Unknown Object (File)
Apr 2 2025, 4:34 AM
Unknown Object (File)
Mar 23 2025, 1:27 PM
Subscribers
None

Details

Summary

Ref T13289. When you create a Phriction document, you currently get an email with the whole new content as a "diff".

You also get extra transactions in the email and on the page.

This is because Phriction isn't on EditEngine and doesn't mark "create" transactions in a modern way. Get them marked properly to fix these obviously-broken behaviors. This can all go away once Phriction switches to EditEngine, although I don't have any particular plans to do that in the immediate future.

Test Plan
  • Created a new document, viewed email, no longer saw redundant "edited content" transaction or "CHANGES TO CONTENT" diff.
  • Updated a document, viewed email, got interdiff.

Diff Detail

Repository
rP Phabricator
Branch
wiki1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 22887
Build 31398: Run Core Tests
Build 31397: arc lint + arc unit