Differential D12405

Conpherence - turn on mentions interface for Conpherence rooms / messages
ClosedPublic
Actions

Authored by btrahan on Apr 13 2015, 11:32 PM.

Tags

None

Referenced Files

	F15577703: D12405.id29801.diff
	Tue, May 6, 3:37 PM

	F15552691: D12405.id.diff
	Sun, Apr 27, 10:05 PM

	F15531464: D12405.id29800.diff
	Wed, Apr 23, 11:20 AM

	F15520325: D12405.diff
	Sun, Apr 20, 5:33 AM

	F15518822: D12405.id29800.diff
	Sat, Apr 19, 5:12 PM

	F15507687: D12405.id.diff
	Tue, Apr 15, 9:58 PM

	F15484305: D12405.id29801.diff
	Wed, Apr 9, 5:32 PM

	F15482326: D12405.id29801.diff
	Wed, Apr 9, 4:34 AM

Subscribers

Details

Reviewers

Maniphest Tasks

T7756: Conpherence threads should support monograms

Commits

Restricted Diffusion Commit
rP295308de5bd8: Conpherence - turn on mentions interface for Conpherence rooms / messages

Summary

Fixes T7756. This is the last little stray bit, though finishing T7757 also helps this feature IMO.

Test Plan

said "ZXX is the best" in comment on DXX and saw proper mention transaction on ZXX

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

btrahan updated this revision to Diff 29799.Apr 13 2015, 11:32 PM

btrahan added a task: T7756: Conpherence threads should support monograms.

btrahan retitled this revision from to Conpherence - turn on mentions interface for Conpherence rooms / messages.

btrahan updated this object.

btrahan edited the test plan for this revision. (Show Details)

btrahan added a reviewer: epriestley.

Herald added subscribers: epriestley, Korvin. · View Herald TranscriptApr 13 2015, 11:32 PM

btrahan added inline comments.Apr 13 2015, 11:32 PM

src/applications/conpherence/view/ConpherenceTransactionView.php
134–136	stray!
src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php
224–226	low level support seemed appropriate for this. other implementers all picked it up for free via project interface.

remove debugging code

epriestley accepted this revision.Apr 14 2015, 12:07 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Apr 14 2015, 12:07 AM

Closed by commit rP295308de5bd8: Conpherence - turn on mentions interface for Conpherence rooms / messages (authored by btrahan, committed by btrahan). · Explain WhyApr 14 2015, 1:12 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

conpherence/

storage/

ConpherenceThread.php

1 line

view/

ConpherenceTransactionView.php

1 line

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

3 lines

Diff 29801

src/applications/conpherence/storage/ConpherenceThread.php

Loading...

src/applications/conpherence/view/ConpherenceTransactionView.php

Loading...

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...