Paths

Table of Contentst

Differential D12651

Users should be able to rsvp to event via email
ClosedPublic
Actions

Authored by lpriestley on May 2 2015, 1:48 AM.

Tags

None

Referenced Files

	F14064106: D12651.diff
	Mon, Nov 18, 9:49 PM

	F14048899: D12651.id30392.diff
	Thu, Nov 14, 10:06 AM

	F14029711: D12651.id30391.diff
	Fri, Nov 8, 8:57 PM

	F14010348: D12651.id30391.diff
	Thu, Oct 31, 7:13 AM

	F13994418: D12651.id30393.diff
	Wed, Oct 23, 6:01 AM

	F13980521: D12651.diff
	Oct 19 2024, 11:12 AM

	F13973735: D12651.id30392.diff
	Oct 18 2024, 2:19 AM

	F13959459: D12651.id30391.diff
	Oct 14 2024, 7:30 PM

Subscribers

Details

Reviewers

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T7957: Implement mail in Calendar, with reply handling

Commits

Restricted Diffusion Commit
rP46a1b816ed97: Users should be able to rsvp to event via email

Summary

Closes T7957, Users should be able to rsvp to event via email.

Test Plan

Create event, open invitee email, reply to email w/ !rsvp yes, refresh event, invitee should show as attending.

Diff Detail

Repository

Branch

calendareventemailrsvp

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 5667
Build 5686: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

lpriestley updated this revision to Diff 30391.May 2 2015, 1:48 AM

lpriestley retitled this revision from to Users should be able to rsvp to event via email.

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 2 2015, 1:48 AM

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

epriestley accepted this revision.May 2 2015, 1:52 AM

epriestley edited edge metadata.

epriestley added inline comments.

src/applications/calendar/command/PhabricatorCalendarEventRSVPEmailCommand.php
86	ya yeah yuh uhuh ok okay yiss aww yiss attend intend to attend confirm confirmed bringing dessert bringing desert time2business time4business
94	nah nuh huh wut no way nuhuh decline declined leave cancel
src/applications/calendar/editor/PhabricatorCalendarEventEditor.php
150	Double check that creating a new event still works? If it complains here, add: ->attachAttendees(array()) ...to `PhabricatorCalendareEvent::initializeNewCalendarEvent()`.

This revision is now accepted and ready to land.May 2 2015, 1:52 AM

Adding valuable rsvp options

lpriestley marked 2 inline comments as done.May 2 2015, 2:00 AM

lpriestley added inline comments.

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php
150	Seems to work okay.

some of those options are real bad

Closed by commit rP46a1b816ed97: Users should be able to rsvp to event via email (authored by lpriestley, committed by lpriestley). · Explain WhyMay 2 2015, 2:01 AM

This revision was automatically updated to reflect the committed changes.

lpriestley marked an inline comment as done.

Revision Contents
Changeset List

Path

Size

src/

__phutil_library_map__.php

4 lines

applications/

calendar/

command/

PhabricatorCalendarEventEmailCommand.php

11 lines

PhabricatorCalendarEventRSVPEmailCommand.php

98 lines

editor/

PhabricatorCalendarEventEditor.php

20 lines

storage/

PhabricatorCalendarEvent.php

2 lines

PhabricatorCalendarEventTransaction.php

1 line

Diff 30391

src/__phutil_library_map__.php

Loading...

src/applications/calendar/command/PhabricatorCalendarEventEmailCommand.php

Loading...

src/applications/calendar/command/PhabricatorCalendarEventRSVPEmailCommand.php

Loading...

src/applications/calendar/editor/PhabricatorCalendarEventEditor.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...

src/applications/calendar/storage/PhabricatorCalendarEventTransaction.php

Loading...