Page MenuHomePhabricator

Properly import all-day events in Calendar
ClosedPublic

Authored by epriestley on Oct 30 2016, 3:14 PM.
Tags
None
Referenced Files
F15590391: D16770.id40396.diff
Fri, May 9, 7:06 PM
F15589861: D16770.diff
Fri, May 9, 3:40 PM
F15582617: D16770.id40392.diff
Wed, May 7, 5:13 PM
F15549678: D16770.id40396.diff
Sun, Apr 27, 7:15 AM
F15547644: D16770.id40392.diff
Sat, Apr 26, 7:55 PM
F15544373: D16770.id40392.diff
Sat, Apr 26, 4:32 AM
F15541671: D16770.id.diff
Fri, Apr 25, 12:57 PM
F15539011: D16770.diff
Thu, Apr 24, 10:48 PM
Subscribers
None

Details

Summary

Ref T10747. The transaction version of this copies the "all day" flag over properly, but this non-transaction version needs to copy it explicitly.

Test Plan

Imported an all-day event, saw it come in as all-day.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

epriestley retitled this revision from to Properly import all-day events in Calendar.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Oct 30 2016, 3:15 PM
This revision was automatically updated to reflect the committed changes.