Differential D12149

Conpherence - implement join / view rules for rooms
ClosedPublic
Actions

Authored by btrahan on Mar 24 2015, 11:25 PM.

Tags

None

Referenced Files

	F13139132: D12149.diff
	Fri, May 3, 2:32 AM

	Unknown Object (File)
	Tue, Apr 30, 4:57 PM

	Unknown Object (File)
	Tue, Apr 30, 4:57 PM

	Unknown Object (File)
	Tue, Apr 30, 4:57 PM

	Unknown Object (File)
	Mon, Apr 29, 2:25 PM

	Unknown Object (File)
	Mon, Apr 29, 2:05 PM

	Unknown Object (File)
	Wed, Apr 24, 10:09 PM

	Unknown Object (File)
	Sun, Apr 21, 4:20 PM

Subscribers

Details

Reviewers

epriestley
chad

Maniphest Tasks

T7585: Implement view/join rules for rooms

Commits

Restricted Diffusion Commit
rP25767096c931: Conpherence - implement join / view rules for rooms

Summary

Ref T7585. This implements everything specified, with a few caveats

since rooms you have yet to join can't be viewed in the column yet, the column view has some bugs and isn't expected to work.
the room you're looking at is just pre-pending to the top of the "recent" list

Test Plan

made a room that no one could join. verified when viewing that there was no comment ui. made a room that others could join. verified folks who had yet to join had a "join" button with an area for text. tried joining with / without message text and it worked in both cases

Diff Detail

Repository

Branch

T7585

Lint

Lint Passed

Unit

No Test Coverage

Build Status

Buildable 4966
Build 4984: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

btrahan updated this revision to Diff 29207.Mar 24 2015, 11:25 PM

btrahan added a task: T7585: Implement view/join rules for rooms.

btrahan retitled this revision from to Conpherence - implement join / view rules for rooms.

btrahan updated this object.

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

btrahan added a reviewer: epriestley.

Herald added a reviewer: chad. · View Herald TranscriptMar 24 2015, 11:25 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

btrahan updated this object.Mar 24 2015, 11:27 PM

btrahan edited edge metadata.

epriestley accepted this revision.Mar 24 2015, 11:45 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Mar 24 2015, 11:45 PM

Closed by commit rP25767096c931: Conpherence - implement join / view rules for rooms (authored by btrahan, committed by btrahan). · Explain WhyMar 25 2015, 1:38 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

celerity/

24 lines

src/

applications/

conpherence/

constants/

ConpherenceUpdateActions.php

1 line

controller/

ConpherenceListController.php

57 lines

ConpherenceUpdateController.php

55 lines

ConpherenceViewController.php

29 lines

ConpherenceWidgetController.php

20 lines

editor/

ConpherenceEditor.php

13 lines

storage/

ConpherenceThread.php

7 lines

transactions/

editor/

PhabricatorApplicationTransactionEditor.php

4 lines

webroot/

rsrc/

css/

application/

conpherence/

widget-pane.css

12 lines

js/

application/

conpherence/

behavior-menu.js

18 lines

Diff 29207

resources/celerity/map.php

Loading...

src/applications/conpherence/constants/ConpherenceUpdateActions.php

Loading...

src/applications/conpherence/controller/ConpherenceListController.php

Loading...

src/applications/conpherence/controller/ConpherenceUpdateController.php

Loading...

src/applications/conpherence/controller/ConpherenceViewController.php

Loading...

src/applications/conpherence/controller/ConpherenceWidgetController.php

Loading...

src/applications/conpherence/editor/ConpherenceEditor.php

Loading...

src/applications/conpherence/storage/ConpherenceThread.php

Loading...

src/applications/transactions/editor/PhabricatorApplicationTransactionEditor.php

Loading...

webroot/rsrc/css/application/conpherence/widget-pane.css

Loading...

webroot/rsrc/js/application/conpherence/behavior-menu.js

Loading...