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

Subscribers

Details

Reviewers

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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

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.

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

Path

Size

src/

applications/

calendar/

controller/

PhabricatorCalendarEventEditController.php

19 lines

PhabricatorCalendarEventViewController.php

5 lines

editor/

PhabricatorCalendarEventEditor.php

2 lines

query/

PhabricatorCalendarEventSearchEngine.php

2 lines

storage/

PhabricatorCalendarEvent.php

45 lines

Diff 30247

src/applications/calendar/controller/PhabricatorCalendarEventEditController.php

Loading...

src/applications/calendar/controller/PhabricatorCalendarEventViewController.php

Loading...

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...

src/applications/calendar/query/PhabricatorCalendarEventSearchEngine.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...