Page MenuHomePhabricator

Make more of the Calendar export workflow work
ClosedPublic

Authored by epriestley on Oct 5 2016, 11:44 PM.
Tags
None
Referenced Files
F14393698: D16676.id40154.diff
Sun, Dec 22, 12:34 AM
F14393697: D16676.id40139.diff
Sun, Dec 22, 12:34 AM
F14393643: D16676.id.diff
Sun, Dec 22, 12:27 AM
Unknown Object (File)
Tue, Dec 17, 4:34 PM
Unknown Object (File)
Wed, Dec 11, 9:33 PM
Unknown Object (File)
Mon, Dec 9, 11:19 AM
Unknown Object (File)
Fri, Dec 6, 7:22 PM
Unknown Object (File)
Fri, Dec 6, 3:17 AM
Subscribers

Details

Summary

Ref T10747.

  • Adds a "Use Results..." dropdown to query result pages, with actions you can take with search results (today: create export; in future: bulk edit, export as excel, make dashboard panel, etc).
  • Allows you to create an export against a query key.
    • I'm just using a text edit field for this for now.
  • Fleshes out export modes. I plan to support: public (as though you were logged out), privileged (as though you were logged in) and availability (event times, but not details).

This does not actually export stuff yet.

Test Plan

Created some exports. Viewed and listed exports.

Diff Detail

Repository
rP Phabricator
Branch
cal13
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 14030
Build 18200: Run Core Tests
Build 18199: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Make more of the Calendar export workflow work.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.

ffs this is a big diff

I can pull the action part out if you want to fiddle with that more or whatever, it just had to sort of do something and I kept going for a bit.

chad edited edge metadata.
This revision is now accepted and ready to land.Oct 6 2016, 12:36 AM
This revision was automatically updated to reflect the committed changes.
vrana added inline comments.
src/applications/calendar/editor/PhabricatorCalendarExportEditEngine.php
73

I'm not sure what should happen here but array_shift accepts only the array to be shifted.

src/applications/calendar/editor/PhabricatorCalendarExportEditEngine.php
73

I think this is supposed to be array_unshift().