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

	Unknown Object (File)
	Fri, May 3, 11:38 PM

	Unknown Object (File)
	Fri, May 3, 11:38 PM

	Unknown Object (File)
	Fri, May 3, 11:38 PM

	Unknown Object (File)
	Fri, May 3, 11:35 PM

	Unknown Object (File)
	Fri, May 3, 11:35 PM

	Unknown Object (File)
	Fri, May 3, 5:23 AM

	Unknown Object (File)
	Thu, Apr 25, 12:52 AM

	Unknown Object (File)
	Tue, Apr 16, 1:47 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

Branch

calendareventemails

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5654
Build 5673: [Placeholder Plan] Wait for 30 Seconds

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

100 lines

mail/

PhabricatorCalendarEventMailReceiver.php

28 lines

PhabricatorCalendarReplyHandler.php

15 lines

storage/

PhabricatorCalendarEvent.php

26 lines

Diff 30370

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