GitHub PushEvent at 2014-02-07 06:12:59
Repository: phabricator
Actor: mbialon
Payload: {"shas":[["ab636f36bf7469fcd79adc18d83fff3ec1b57e5a","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Rename PhabricatorUserStatus to PhabricatorCalendarEvent\n\nSummary: Ref T4375. We never join this table, so this is a pretty straight find/replace.\n\nTest Plan: Browsed around Calendar, verified that nothing seemed broken. Saw my red dot in other apps.\n\nReviewers: btrahan, chad\n\nReviewed By: btrahan\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8145","epriestley",true],["258cf2368482a397ed9aa47b729a2d516e145c1f","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Use a modern query object to query Calendar events\n\nSummary: Ref T4375. Calendar uses oldschool `loadOneWhere()` calls. Make CalendarEvent policy-aware, do the edit/delete policy checks through the policy framework, and use modern query infrastructure.\n\nTest Plan:\n  - Viewed calendar;\n  - created, edited, deleted event;\n  - viewed calendar tab in Conpherence.\n\nReviewers: btrahan, chad\n\nReviewed By: btrahan\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8146","epriestley",true],["8410cbecb08a1a057636e420502a30b9704e0ec1","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Rename \"status\" to \"event\" in most URIs / classes / etc\n\nSummary: Ref T4375. This doesn't get everything (I figure I'll clean up the actual UI strings when I touch the UIs) but should get the bulk of the URIs and class names and stuff.\n\nTest Plan: Clicked every calendar-related link I could find/grep, they all still seem to work. URIs now say \"event\".\n\nReviewers: btrahan, chad\n\nReviewed By: btrahan\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8147","epriestley",true],["0ae0f352b05361c464679536fc1c4c68d353628f","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Use ApplicationSearch in the calendar event list view\n\nSummary: Ref T4375. Basic ApplicationSearch integration to power this more flexibly.\n\nTest Plan: {F108762}\n\nReviewers: btrahan, chad\n\nReviewed By: btrahan\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8148","epriestley",true],["b8fce80b05d030314f4db2e1b54bfb743a2b7b09","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Give Calendar events a more modern view/detail page\n\nSummary: Ref T4375. Very basic, but gives us a more standard place to put edit/delete operations.\n\nTest Plan: {F108765}\n\nReviewers: btrahan, chad\n\nReviewed By: btrahan\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8149","epriestley",true],["4c7acbaface49583c72062b5f2dd906d97a21684","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Assign PHIDs to calendar events\n\nSummary: Ref T4375. We're going to need these for a bunch of infrastructure to work.\n\nTest Plan: Ran migrations, checked DB, used `phid.query`.\n\nReviewers: btrahan, chad\n\nReviewed By: chad\n\nCC: aran\n\nManiphest Tasks: T4375\n\nDifferential Revision: https://secure.phabricator.com/D8151","epriestley",true],["eef2727ea6eefd5ddda75dce99c7b61d73b7b250","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Fix accidental forward dependency in old commit summary migration\n\nSummary: See IRC. This migration inadvertently depends on the columns in the commit table, because it calls `save()`, and thus broke for installs with data after we added the `importStatus` column. Since that was ~9 months after this patch, probably not many installs are affected.\n\nTest Plan: Ran patch locally with `--apply` on data. Had user verify fix.\n\nReviewers: btrahan\n\nReviewed By: btrahan\n\nCC: aran\n\nDifferential Revision: https://secure.phabricator.com/D8152","epriestley",true],["a2687caa332411f30bb571c463fffe031e1981fd","46f1a0bd5592a2f9244ca321b129902a06b53e03@epriestley.com","Fix escaped HTML in burnup report\n\nSummary: Fixes T4378. I just removed the `<em>` since this element is unusual and it's not convenient to switch it to translatable remarkup.\n\nTest Plan: stared at it real good\n\nReviewers: btrahan\n\nReviewed By: btrahan\n\nCC: aran, chrisbolt\n\nManiphest Tasks: T4378\n\nDifferential Revision: https://secure.phabricator.com/D8155","epriestley",true],["1527a967c07529b73b738f2061d2a07c477798b5","1cc41d42c253533bf101be3014dba14f42b0d67c@gmail.com","Herald - add support for task priority\n\nSummary: adds a new FIELD and a new VALUE to support this. Slightly dodgy because priorities do not have phids so we have to special case how we handle this in a few spots. Ref T4294.\n\nTest Plan: made a new rule to get cc'd on unbreak now and wishlist tasks. verified got cc'd correctly and not cc'd correctly.\n\nReviewers: epriestley\n\nReviewed By: epriestley\n\nCC: Korvin, epriestley, aran\n\nManiphest Tasks: T4294\n\nDifferential Revision: https://secure.phabricator.com/D8156","Bob Trahan",true],["a4871d034a30a5a2326330715947dced48141d74","1cc41d42c253533bf101be3014dba14f42b0d67c@gmail.com","Herald - unleash some recursive madness!\n\nSummary: turn on herald rules ability to specify other herald rules. Fixes T4294.\n\nTest Plan: made a rule to be cc'd on new tasks. made another rule to flag a task if it contained \"test test\" in the title AND the cc'd rule for new tasks matched. Made some new tasks and verified new \"test test\" tasks were flagged.\n\nReviewers: epriestley\n\nReviewed By: epriestley\n\nCC: Korvin, epriestley, aran\n\nManiphest Tasks: T4294\n\nDifferential Revision: https://secure.phabricator.com/D8157","Bob Trahan",true],["0cc1f50170832d2d6b95f0df294e264e4866138b","1cc41d42c253533bf101be3014dba14f42b0d67c@gmail.com","Herald - make flag color display\n\nSummary: It used to say \"Mark with flag 7\" or whatever, and now it says \"Mark with flag Checkered\"\n\nTest Plan: noted previous rule I made was more understandable\n\nReviewers: epriestley\n\nReviewed By: epriestley\n\nCC: Korvin, epriestley, aran\n\nDifferential Revision: https://secure.phabricator.com/D8158","Bob Trahan",true],["f6f800c15244d131c77de01cabb0cdcd15d89d15","75d547cc96937d13b87cf614e50d1ea059d60c0a@bialon.net","Merge remote-tracking branch 'upstream/master' into release","Marcin Białoń",true]],"size":12,"ref":"refs/heads/release","head":"f6f800c15244d131c77de01cabb0cdcd15d89d15"}