Differential D14839

repository.query now includes project PHIDs
AbandonedPublic
Actions

Authored by dereckson on Dec 21 2015, 4:04 PM.

Details

Reviewers

None

Group Reviewers

Blessed Reviewers

Maniphest Tasks

T10030: repository.query API should include a projectPHIDs field

Summary

Add a toDictionaryWithProjects::toDictionaryWithProjects to get a dictionary with the properties of the object, including the attached projects
Conduit repository.query API reply now includes projects PHIDs

Test Plan

Test through /conduit/method/repository.query/

Diff Detail

Repository

rP Phabricator

Lint

Lint Skipped

Unit

Tests Skipped

Event Timeline

dereckson updated this revision to Diff 35869.Dec 21 2015, 4:04 PM

dereckson retitled this revision from to repository.query now includes project PHIDs.

dereckson updated this object.

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

dereckson set the repository for this revision to rP Phabricator.

dereckson added projects: Conduit, Diffusion.

Herald added a reviewer: Blessed Reviewers. · View Herald TranscriptDec 21 2015, 4:04 PM

Herald added a subscriber: epriestley. · View Herald Transcript

dereckson added a task: T10030: repository.query API should include a projectPHIDs field.Dec 21 2015, 4:04 PM

I see D10289 has already offered a similar change.

The difference here is we only focus on the reply, not the query.

Per T10030#150337.

Revision Contents
Changeset List

Path

Size

src/

applications/

repository/

conduit/

RepositoryQueryConduitAPIMethod.php

5 lines

storage/

PhabricatorRepository.php

6 lines

Diff 35869

View Options

src/applications/repository/conduit/RepositoryQueryConduitAPIMethod.php