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

	F13981249: D12595.id30247.diff
	Sat, Oct 19, 3:11 PM

	F13971206: D12595.id30246.diff
	Thu, Oct 17, 12:18 PM

	F13970940: D12595.id.diff
	Thu, Oct 17, 10:39 AM

	Unknown Object (File)
	Oct 10 2024, 9:14 PM

	Unknown Object (File)
	Oct 1 2024, 11:06 PM

	Unknown Object (File)
	Oct 1 2024, 4:08 AM

	Unknown Object (File)
	Sep 24 2024, 1:07 PM

	Unknown Object (File)
	Sep 20 2024, 7:34 AM

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

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

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 30246

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...