Paths

Table of Contentst

Differential D16772

Give organizers in ICS exports a dummy email address to placate Gmail
ClosedPublic
Actions

Authored by epriestley on Oct 30 2016, 4:13 PM.

Tags

None

Referenced Files

	F13238188: D16772.id40398.diff
	Tue, May 21, 5:44 PM

	F13238072: D16772.id40394.diff
	Tue, May 21, 4:57 PM

	F13238055: D16772.id.diff
	Tue, May 21, 4:50 PM

	F13213382: D16772.diff
	Fri, May 17, 7:42 AM

	F13209667: D16772.diff
	Fri, May 17, 1:29 AM

	Unknown Object (File)
	Tue, May 7, 6:34 AM

	Unknown Object (File)
	Sun, May 5, 6:25 PM

	Unknown Object (File)
	May 3 2024, 4:03 AM

Subscribers

None

Details

Reviewers

Maniphest Tasks

T10747: Import and export ICS from Calendar

Commits

rPe1a9b7694517: Give organizers in ICS exports a dummy email address to placate Gmail

Summary

Ref T10747. In the in-email ICS event card that Gmail shows, it has a "Who" field which reads "Unknown Organizer*" if the URI for the organizer isn't email-address-like.

Previously, we used a URI like https://phabricator.install.com/p/username, which I think is OK as far as RFC 5545 is concerned, but Gmail doesn't like it.

Instead, use PHID-USER-asdfa@phabricator.install.com, which doesn't go anywhere, but makes Gmail happy. Users don't normally see this URI anyway.

Test Plan

Got a readable "Who" in Gmail when importing an event exported from Calendar:

Screen Shot 2016-10-30 at 9.12.16 AM.png (272×525 px, 25 KB)

Diff Detail

Repository

Lint

Lint Not Applicable

Unit

Tests Not Applicable

Event Timeline

epriestley updated this revision to Diff 40394.Oct 30 2016, 4:13 PM

epriestley retitled this revision from to Give organizers in ICS exports a dummy email address to placate Gmail.

epriestley updated this object.

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

epriestley added a reviewer: chad.

epriestley added a task: T10747: Import and export ICS from Calendar.

chad accepted this revision.Oct 30 2016, 4:18 PM

chad edited edge metadata.

This revision is now accepted and ready to land.Oct 30 2016, 4:18 PM

Closed by commit rPe1a9b7694517: Give organizers in ICS exports a dummy email address to placate Gmail (authored by epriestley, committed by epriestley). · Explain WhyOct 30 2016, 4:21 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

storage/

PhabricatorCalendarEvent.php

14 lines

Diff 40398

src/applications/calendar/storage/PhabricatorCalendarEvent.php

Loading...