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
F13283005: D16676.diff
Sun, Jun 2, 12:42 PM
F13278595: D16676.diff
Fri, May 31, 11:29 PM
F13271067: D16676.id40154.diff
Wed, May 29, 4:06 PM
F13267992: D16676.diff
Wed, May 29, 4:29 AM
F13253474: D16676.diff
Sat, May 25, 2:42 AM
F13232068: D16676.diff
Tue, May 21, 1:00 AM
F13220099: D16676.diff
Sat, May 18, 11:13 PM
F13210939: D16676.diff
Fri, May 17, 5:20 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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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().