Differential D13415

Set event policies like other event properties
ClosedPublic
Actions

Authored by lpriestley on Jun 24 2015, 9:52 PM.

Details

Reviewers

epriestley

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T8028: Fix PolicyControl defaulting nonsense in every application

Commits

Restricted Diffusion Commit
rPb1a843b21a38: Set event policies like other event properties

Summary

Closes T8028, Set event policies like other event properties

Test Plan

Create and edit events, make sure edit and view policies get populated and saved correctly.

Diff Detail

Repository

rP Phabricator

Branch

calendarpolicies

Lint

Lint Passed

Unit

Tests Passed

Build Status

Buildable 6978
Build 7015: [Placeholder Plan] Wait for 30 Seconds
Build 7014: arc lint + arc unit

Event Timeline

lpriestley updated this revision to Diff 32493.Jun 24 2015, 9:52 PM

lpriestley retitled this revision from to Set event policies like other event properties.

lpriestley updated this object.

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

lpriestley added a reviewer: epriestley.

lpriestley added a task: T8028: Fix PolicyControl defaulting nonsense in every application.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptJun 24 2015, 9:52 PM

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

epriestley accepted this revision.Jun 26 2015, 3:50 AM

epriestley edited edge metadata.

This revision is now accepted and ready to land.Jun 26 2015, 3:50 AM

Closed by commit rPb1a843b21a38: Set event policies like other event properties (authored by lpriestley, committed by lpriestley). · Explain WhyJun 26 2015, 5:41 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

src/

applications/

calendar/

controller/

PhabricatorCalendarController.php

2 lines

PhabricatorCalendarEventEditController.php

17 lines

Diff 32493

View Options

src/applications/calendar/controller/PhabricatorCalendarController.php