Page MenuHomePhabricator

Modernize Conpherence with Modular Transactions
ClosedPublic

Authored by chad on Apr 12 2017, 10:02 PM.
Tags
None
Referenced Files
F19543750: D17668.id42491.diff
Thu, Jan 22, 11:27 PM
F19034723: D17668.id.diff
Nov 25 2025, 2:43 PM
F18821716: D17668.id.diff
Oct 23 2025, 1:27 AM
F18816513: D17668.diff
Oct 21 2025, 7:25 AM
F18804020: D17668.id42495.diff
Oct 18 2025, 5:32 AM
F18780716: D17668.id42491.diff
Oct 12 2025, 7:06 PM
F18468378: D17668.id.diff
Sep 2 2025, 3:09 PM
F18454599: D17668.diff
Sep 1 2025, 5:19 AM
Subscribers

Details

Summary

Begin converting Conpherence to ModularTransactions, this converts title, topic, and picture to use modular transactions. Participants seems hairy so I'll do that in another diff

Test Plan

Create a room with a topic, change room name, topic. Add people, remove people. Set a room image. Unset topic.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley added inline comments.
src/applications/conpherence/editor/ConpherenceEditor.php
36

We'll want a getObjectCreateString() or whatever it is here, for "%s created this room.", although we won't actually start hitting it until we convert to EditEngine.

544–559

This means that we'll never publish Conpherence feed stories. I think that's actually good since they always seem weird to me, but wanted to make sure it's intentional.

This revision is now accepted and ready to land.Apr 12 2017, 10:07 PM

No I love feed stories. And sending emails.

This revision was automatically updated to reflect the committed changes.