Page MenuHomePhabricator
Differential D8660

Support querying a specific snapshot on phragment.queryfragments
AbandonedPublic
Actions

Authored by hach-que on Apr 1 2014, 1:03 AM.
Tags
None
Referenced Files
F13842480: D8660.diff
Thu, Sep 26, 9:40 PM
Unknown Object (File)
Thu, Sep 12, 9:20 AM
Unknown Object (File)
Thu, Sep 12, 9:20 AM
Unknown Object (File)
Fri, Sep 6, 10:51 AM
Unknown Object (File)
Aug 26 2024, 11:24 AM
Unknown Object (File)
Aug 21 2024, 4:04 AM
Unknown Object (File)
Aug 8 2024, 10:37 AM
Unknown Object (File)
Aug 2 2024, 12:56 PM
View All Files
Subscribers
epriestley
hach-que
joshuaspence
Korvin

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Maniphest Tasks
T5479: Unbeta Phragment
T4884: At some point, review hach-que's diffs
Summary

This allows a snapshot name to be specified when querying for Phragment fragments via Conduit. It also adds the "size" field onto the reported results, which is the size of the file in bytes.

Test Plan

Specified snapshot, saw the data change.

Diff Detail

Repository
rP Phabricator
Branch
phragment-conduit-snapshot
Lint
Lint Errors
SeverityLocationCodeMessage
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:41PHL1Unknown Symbol
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:49PHL1Unknown Symbol
Errorsrc/infrastructure/util/password/PhabricatorBcryptPasswordHasher.php:53PHL1Unknown Symbol
Unit
No Test Coverage

Event Timeline

hach-que updated this revision to Diff 20538.Apr 1 2014, 1:03 AM
hach-que retitled this revision from to Support querying a specific snapshot on phragment.queryfragments.
hach-que updated this object.
hach-que edited the test plan for this revision. (Show Details)
hach-que added a reviewer: epriestley.
Herald added a reviewer: Blessed Reviewers.Apr 1 2014, 1:03 AM
Herald added subscribers: hach-que, Korvin, epriestley.
hach-que updated this revision to Diff 24552.Aug 9 2014, 5:17 AM
hach-que edited edge metadata.

Rebase on master

Harbormaster completed remote builds in B2126: Diff 24552.Aug 9 2014, 5:18 AM
joshuaspence added a subscriber: joshuaspence.Mar 1 2015, 7:18 AM
epriestley requested changes to this revision.May 13 2015, 1:56 PM
epriestley edited edge metadata.

This should wait for T7715 and be rebuilt on top of ApplicationSearch-via-Conduit.

This revision now requires changes to proceed.May 13 2015, 1:56 PM
hach-que mentioned this in D8669: Add phragment.querysnapshots method.Jun 18 2015, 1:28 PM
hach-que abandoned this revision.Jun 18 2015, 2:11 PM

Might as well abandon this as we're no longer using it in production (and is probably still years away on the upstream roadmap).

Revision Contents
Changeset List

PathSize
src/
applications/
phragment/
conduit/
82 lines

Diff 20538

View Options

src/applications/phragment/conduit/ConduitAPI_phragment_queryfragments_Method.php

Loading...
Phabricator · Built by Phacility