Page MenuHomePhabricator

Fix an issue with "packages(...)" in typeaheads
ClosedPublic

Authored by epriestley on Sep 1 2015, 11:17 AM.

Details

Summary

Fixes T9302. This datasource wasn't resolving package PHIDs correctly for the actual query.

Also fixes an issue with the "Affected packages that need audit" Herald rule.

Test Plan

Ran a "Needs Audit" query with only packages, and only packages(user).

Diff Detail

Repository
rP Phabricator
Branch
auditfix1
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 7871
Build 8773: [Placeholder Plan] Wait for 30 Seconds
Build 8772: arc lint + arc unit

Event Timeline

epriestley retitled this revision from to Fix an issue with "packages(...)" in typeaheads.
epriestley updated this object.
epriestley edited the test plan for this revision. (Show Details)
epriestley added a reviewer: chad.
chad edited edge metadata.
This revision is now accepted and ready to land.Sep 1 2015, 3:17 PM
This revision was automatically updated to reflect the committed changes.