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

	F14376337: D12651.diff
	Fri, Dec 20, 11:38 PM

	Unknown Object (File)
	Mon, Dec 16, 8:30 PM

	Unknown Object (File)
	Nov 18 2024, 9:49 PM

	Unknown Object (File)
	Nov 14 2024, 10:06 AM

	Unknown Object (File)
	Nov 8 2024, 8:57 PM

	Unknown Object (File)
	Oct 31 2024, 7:13 AM

	Unknown Object (File)
	Oct 23 2024, 6:01 AM

	Unknown Object (File)
	Oct 19 2024, 11:12 AM

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

Lint

Lint Not Applicable

Unit

Tests Not Applicable

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
87	ya yeah yuh uhuh ok okay yiss aww yiss attend intend to attend confirm confirmed bringing dessert bringing desert time2business time4business
95	nah nuh huh wut no way nuhuh decline declined leave cancel
src/applications/calendar/editor/PhabricatorCalendarEventEditor.php
150–151	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–151	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

124 lines

editor/

PhabricatorCalendarEventEditor.php

20 lines

storage/

PhabricatorCalendarEvent.php

2 lines

PhabricatorCalendarEventTransaction.php

1 line

Diff 30393

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