Paths

Table of Contentst

Differential D12640

Calendar events should support comments
ClosedPublic
Actions

Authored by lpriestley on May 1 2015, 12:25 AM.

Tags

None

Referenced Files

	F13258959: D12640.id30352.diff
	Sun, May 26, 2:27 PM

	F13232853: D12640.diff
	Tue, May 21, 1:31 AM

	F13231853: D12640.id30352.diff
	Tue, May 21, 12:49 AM

	F13231851: D12640.id30351.diff
	Tue, May 21, 12:49 AM

	F13231849: D12640.id30350.diff
	Tue, May 21, 12:49 AM

	F13219857: D12640.id30350.diff
	Sat, May 18, 9:29 PM

	F13213508: D12640.diff
	Fri, May 17, 7:50 AM

	F13206315: D12640.id.diff
	Wed, May 15, 7:05 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T7956: Support commenting on Calendar events

Commits

Restricted Diffusion Commit
rPbbba6241c2a1: Calendar events should support comments

Summary

Closes T7956, Calendar events should support comments.

Test Plan

Open event, add comment, save, comment should appear in timeline.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30350.May 1 2015, 12:25 AM

lpriestley retitled this revision from to Calendar events should support comments.

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T7956: Support commenting on Calendar events.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 1 2015, 12:25 AM

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

Use getMonogram!

epriestley accepted this revision.May 1 2015, 12:31 AM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/calendar/application/PhabricatorCalendarApplication.php
59	indent for consistncyreys
src/applications/calendar/controller/PhabricatorCalendarEventCommentController.php
7–14	Consider modernizing to use `handleRequest(AphrontRequest $request)` instead.
11	Stray debugging code.

This revision is now accepted and ready to land.May 1 2015, 12:31 AM

Considered modernizing it.

Closed by commit rPbbba6241c2a1: Calendar events should support comments (authored by lpriestley, committed by lpriestley). · Explain WhyMay 1 2015, 12:38 AM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

2 lines

applications/

calendar/

application/

PhabricatorCalendarApplication.php

2 lines

controller/

PhabricatorCalendarEventCommentController.php

69 lines

PhabricatorCalendarEventViewController.php

15 lines

editor/

PhabricatorCalendarEventEditor.php

3 lines

Diff 30353

src/__phutil_library_map__.php

Loading...

src/applications/calendar/application/PhabricatorCalendarApplication.php

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventCommentController.php

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...