Page MenuHomePhabricator
Differential D13481

Commenting on instances of a recurring event should create a new event with its own comments
ClosedPublic
Actions

Authored by lpriestley on Jun 29 2015, 10:41 PM.
Tags
None
Referenced Files
Unknown Object (File)
Thu, Feb 20, 9:22 PM
Unknown Object (File)
Sat, Feb 8, 4:53 PM
Unknown Object (File)
Sat, Feb 8, 4:52 PM
Unknown Object (File)
Sat, Feb 8, 4:52 PM
Unknown Object (File)
Sat, Feb 8, 4:51 PM
Unknown Object (File)
Sat, Feb 8, 4:51 PM
Unknown Object (File)
Fri, Jan 31, 1:15 PM
Unknown Object (File)
Wed, Jan 29, 4:42 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T8696: User comments in recurring events should be added per instance
Commits
Restricted Diffusion Commit
rP28e0af5a62c8: Commenting on instances of a recurring event should create a new event with…
Summary

Closes T8696, Commenting on instances of a recurring event should create a new event with its own comments

Test Plan

Open recurring event instance (not exception), should see no timeline, leave a comment, an exception to event should be created.

Diff Detail

Repository
rP Phabricator
Branch
calendarrecurringcomments
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7067
Build 7187: [Placeholder Plan] Wait for 30 Seconds
Build 7186: arc lint + arc unit

Event Timeline

lpriestley updated this revision to Diff 32619.Jun 29 2015, 10:41 PM
lpriestley retitled this revision from to Commenting on instances of a recurring event should create a new event with its own comments.
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T8696: User comments in recurring events should be added per instance.
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 29 2015, 10:41 PM
Herald added subscribers: Korvin, epriestley. · View Herald Transcript
epriestley accepted this revision.Jun 29 2015, 10:51 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/controller/PhabricatorCalendarEventViewController.php
97

Sometimes no $sequence in this

This revision is now accepted and ready to land.Jun 29 2015, 10:51 PM
lpriestley updated this revision to Diff 32638.Jun 30 2015, 6:47 PM
lpriestley edited edge metadata.

$sequence only when it exists

Closed by commit rP28e0af5a62c8: Commenting on instances of a recurring event should create a new event with… (authored by lpriestley, committed by lpriestley). · Explain WhyJun 30 2015, 6:47 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 32638

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...
Phabricator · Built by Phacility