Page MenuHomePhabricator

Set time and date on Calendar Date Control form
ClosedPublic

Authored by chad on Apr 9 2016, 6:52 PM.
Tags
None
Referenced Files
F18803221: D15666.id.diff
Fri, Oct 17, 10:13 PM
F18790429: D15666.id37754.diff
Wed, Oct 15, 2:24 PM
F18783846: D15666.diff
Mon, Oct 13, 10:05 AM
F18737342: D15666.id.diff
Wed, Oct 1, 10:15 AM
F18730641: D15666.diff
Tue, Sep 30, 1:25 PM
F18646866: D15666.id.diff
Sep 19 2025, 11:20 AM
F18103480: D15666.id37754.diff
Aug 10 2025, 5:47 AM
F18087817: D15666.id37754.diff
Aug 6 2025, 5:39 AM
Tokens
"Like" token, awarded by CodeMouse92.

Details

Summary

Recurring events will fatal a Calendar with this not set. newDateTime requires a date and time to be called property. I think this is correct fix? Fixes T10766

Test Plan

Build a recurring event, pull up /calendar/, see recurring events as expected. Previously, fatal.

Diff Detail

Repository
rP Phabricator
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

chad retitled this revision from to Set time and date on Calendar Date Control form.
chad updated this object.
chad edited the test plan for this revision. (Show Details)
chad added reviewers: epriestley, lpriestley.
This revision is now accepted and ready to land.Apr 9 2016, 6:56 PM
This revision was automatically updated to reflect the committed changes.