Page MenuHomePhabricator

Modernize Conpherence with Modular Transactions
ClosedPublic

Authored by chad on Apr 12 2017, 10:02 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Jan 29, 11:26 PM
Unknown Object (File)
Wed, Jan 22, 12:27 PM
Unknown Object (File)
Jan 17 2025, 11:32 PM
Unknown Object (File)
Dec 30 2024, 10:55 PM
Unknown Object (File)
Dec 9 2024, 5:00 PM
Unknown Object (File)
Dec 5 2024, 11:13 PM
Unknown Object (File)
Nov 28 2024, 4:46 AM
Unknown Object (File)
Nov 26 2024, 7:13 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
Branch
conpherence-xaction (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 16422
Build 21850: Run Core Tests
Build 21849: arc lint + arc unit

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.