Page MenuHomePhabricator

Transactions - make TYPE_COMMENT implementation optional
ClosedPublic

Authored by btrahan on May 19 2015, 7:31 PM.
Tags
None
Referenced Files
Unknown Object (File)
Sun, Mar 10, 6:29 AM
Unknown Object (File)
Feb 8 2024, 12:15 AM
Unknown Object (File)
Feb 2 2024, 10:57 AM
Unknown Object (File)
Jan 22 2024, 6:47 PM
Unknown Object (File)
Dec 28 2023, 4:00 AM
Unknown Object (File)
Dec 27 2023, 2:45 AM
Unknown Object (File)
Dec 18 2023, 6:49 PM
Unknown Object (File)
Dec 18 2023, 6:49 PM
Subscribers
Tokens
"Haypence" token, awarded by epriestley.

Details

Summary

Ref T6403. Conpherence keeps track of comments for message counts so we needed some special attention there. Otherwise, straight-forward.

Test Plan

left a comment on a diff with inline comments. sent messages in conpherence successfully. verified unread count incremented correctly for sent messages for users.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

btrahan retitled this revision from to Transactions - make TYPE_COMMENT implementation optional.
btrahan updated this object.
btrahan edited the test plan for this revision. (Show Details)
btrahan added a reviewer: epriestley.
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/conpherence/editor/ConpherenceEditor.php
303

Maybe just break; in case the parent ever does anything.

This revision is now accepted and ready to land.May 19 2015, 7:32 PM
btrahan edited edge metadata.

go with break; rather than return; so the parent can be fancy maybe some day

This revision was automatically updated to reflect the committed changes.