Page MenuHomePhabricator
Differential D12640

Calendar events should support comments
ClosedPublic
Actions

Authored by lpriestley on May 1 2015, 12:25 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 12 2024, 8:51 PM
Unknown Object (File)
Nov 27 2024, 3:20 PM
Unknown Object (File)
Nov 19 2024, 1:36 AM
Unknown Object (File)
Nov 12 2024, 7:30 AM
Unknown Object (File)
Oct 23 2024, 3:41 PM
Unknown Object (File)
Oct 19 2024, 10:12 PM
Unknown Object (File)
Oct 15 2024, 9:29 PM
Unknown Object (File)
Oct 11 2024, 10:05 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
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
rP Phabricator
Branch
calendareventcomments
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5640
Build 5659: [Placeholder Plan] Wait for 30 Seconds

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
lpriestley updated this revision to Diff 30351.May 1 2015, 12:29 AM
lpriestley edited edge metadata.

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
lpriestley updated this revision to Diff 30352.May 1 2015, 12:37 AM
lpriestley marked 3 inline comments as done.
lpriestley edited edge metadata.

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

Diff 30350

View Options

src/__phutil_library_map__.php

Loading...
View Options

src/applications/calendar/application/PhabricatorCalendarApplication.php

Loading...
View Options

src/applications/calendar/controller/PhabricatorCalendarEventCommentController.php

Loading...
View Options

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...
View Options

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...
Phabricator · Built by Phacility