Paths

Table of Contentst

Differential D12645

Calendar event transaction emails without reply handling
ClosedPublic
Actions

Authored by lpriestley on May 1 2015, 6:33 PM.

Tags

None

Referenced Files

	F18811997: D12645.id30377.diff
	Mon, Oct 20, 6:22 AM

	F18670563: D12645.id30368.diff
	Thu, Sep 25, 12:42 AM

	F18648436: D12645.id30379.diff
	Sep 20 2025, 9:36 AM

	F18647312: D12645.id.diff
	Sep 19 2025, 12:49 PM

	F18622280: D12645.id30368.diff
	Sep 15 2025, 11:05 AM

	F18612689: D12645.diff
	Sep 14 2025, 9:45 AM

	F18600217: D12645.id.diff
	Sep 13 2025, 9:14 AM

	F18600193: D12645.id30377.diff
	Sep 13 2025, 9:11 AM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T7957: Implement mail in Calendar, with reply handling

Commits

Restricted Diffusion Commit
rPaa68cc8830d8: Calendar event transaction emails without reply handling

Summary

Ref T7957, Calendar event transaction emails without reply handling.

Test Plan

Create event, invitees should get email.

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

lpriestley updated this revision to Diff 30368.May 1 2015, 6:33 PM

lpriestley retitled this revision from to Calendar event transaction emails without reply handling.

lpriestley updated this object.

lpriestley edited the test plan for this revision. (Show Details)

lpriestley added a reviewer: epriestley.

lpriestley added a task: T7957: Implement mail in Calendar, with reply handling.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptMay 1 2015, 6:33 PM

Herald added subscribers: epriestley, Korvin. · View Herald Transcript

epriestley requested changes to this revision.May 1 2015, 6:44 PM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php
231	For consistency with other applications., prefer `[Calendar]` (no caps).
234–236	Just leave this unimplemented to use the default (the object's PHID). Subclasses which override this do so for historical reasons, to avoid breaking existing threads.
258–279	Remove or convert to be consistent with Calendar.

This revision now requires changes to proceed.May 1 2015, 6:44 PM

Adding reply handler

Cleanup

Cleaning up MAILTAGs

epriestley accepted this revision.May 1 2015, 8:24 PM

epriestley edited edge metadata.

This revision is now accepted and ready to land.May 1 2015, 8:24 PM

lint pht cleanup

Closed by commit rPaa68cc8830d8: Calendar event transaction emails without reply handling (authored by lpriestley, committed by lpriestley). · Explain WhyMay 1 2015, 8:26 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

resources/

sql/

autopatches/

20150501.calendar.1.reply.sql

2 lines

20150501.calendar.2.reply.php

21 lines

src/

__phutil_library_map__.php

4 lines

applications/

calendar/

application/

PhabricatorCalendarApplication.php

14 lines

editor/

PhabricatorCalendarEventEditor.php

88 lines

mail/

PhabricatorCalendarEventMailReceiver.php

28 lines

PhabricatorCalendarReplyHandler.php

15 lines

storage/

PhabricatorCalendarEvent.php

26 lines

PhabricatorCalendarEventTransaction.php

20 lines

Diff 30379

resources/sql/autopatches/20150501.calendar.1.reply.sql

Loading...

resources/sql/autopatches/20150501.calendar.2.reply.php

Loading...

src/__phutil_library_map__.php

Loading...

src/applications/calendar/application/PhabricatorCalendarApplication.php

Loading...

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...

src/applications/calendar/mail/PhabricatorCalendarEventMailReceiver.php

Loading...

src/applications/calendar/mail/PhabricatorCalendarReplyHandler.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php

Loading...