Page MenuHomePhabricator
Differential D12595

Calendar events should support subscribers, for possible future uses.
ClosedPublic
Actions

Authored by lpriestley on Apr 28 2015, 5:17 PM.
Tags
None
Referenced Files
F14352782: D12595.diff
Thu, Dec 19, 2:45 PM
Unknown Object (File)
Tue, Dec 17, 9:52 AM
Unknown Object (File)
Tue, Dec 10, 5:09 AM
Unknown Object (File)
Thu, Dec 5, 4:37 PM
Unknown Object (File)
Wed, Dec 4, 8:15 AM
Unknown Object (File)
Wed, Dec 4, 8:15 AM
Unknown Object (File)
Wed, Dec 4, 8:15 AM
Unknown Object (File)
Wed, Nov 27, 7:02 PM
View All Files
Subscribers
epriestley
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T7974: Subscribers for Calendar events.
Commits
Restricted Diffusion Commit
rPc6fa4355cc92: Calendar events should support subscribers, for possible future uses.
Summary

Closes T7974, Calendar events support subscribers.

Test Plan

Create or update calendar event, add subscribers, save, event detail view should show subscribers, and timeline should reflect the action.

Diff Detail

Repository
rP Phabricator
Branch
calendareventsubscribers
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 5570
Build 5589: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30239.Apr 28 2015, 5:17 PM
lpriestley retitled this revision from to Calendar events should support subscribers, for possible future uses..
lpriestley updated this object.
lpriestley edited the test plan for this revision. (Show Details)
lpriestley added a reviewer: epriestley.
lpriestley added a task: T7974: Subscribers for Calendar events..
Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptApr 28 2015, 5:17 PM
Herald added subscribers: epriestley, Korvin. · View Herald Transcript
lpriestley updated this revision to Diff 30240.Apr 28 2015, 5:20 PM
lpriestley edited edge metadata.

Renaming cc_value to subscribers for consistency.

epriestley accepted this revision.Apr 28 2015, 5:27 PM
epriestley edited edge metadata.
epriestley added inline comments.
src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php
30–31 ↗(On Diff #30240)

Not used?

233–234 ↗(On Diff #30240)

Unintended change?

This revision is now accepted and ready to land.Apr 28 2015, 5:27 PM
lpriestley marked 2 inline comments as done.Apr 28 2015, 5:34 PM
lpriestley added inline comments.
src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php
30–31 ↗(On Diff #30240)

Correct. I was adding and deleting the EDGE events, so that's a remnant of that. Taking it out.

233–234 ↗(On Diff #30240)

Correct.

lpriestley updated this revision to Diff 30246.Apr 28 2015, 5:35 PM
lpriestley marked 2 inline comments as done.
lpriestley edited edge metadata.

Cleaning up transactions.

Closed by commit rPc6fa4355cc92: Calendar events should support subscribers, for possible future uses. (authored by lpriestley, committed by lpriestley). · Explain WhyApr 28 2015, 5:40 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 30246

View Options

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Loading...
View Options

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...
View Options

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...
View Options

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...
View Options

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...
Phabricator · Built by Phacility