Page MenuHomePhabricator

Created Conduit API for Phrequent
Needs RevisionPublic

Authored by emiljt on May 23 2014, 6:14 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Apr 9, 2:26 PM
Unknown Object (File)
Mon, Apr 1, 2:16 PM
Unknown Object (File)
Mar 10 2024, 1:37 AM
Unknown Object (File)
Mar 10 2024, 12:47 AM
Unknown Object (File)
Mar 4 2024, 10:28 PM
Unknown Object (File)
Feb 10 2024, 6:29 AM
Unknown Object (File)
Feb 4 2024, 3:46 AM
Unknown Object (File)
Dec 31 2023, 4:20 PM

Details

Reviewers
epriestley
Group Reviewers
Blessed Reviewers
Summary

Added Conduit API functionality to the Phrequent application. The only method is a basic query to pull Phrequent time tracks.

Test Plan
  • Ran query calls on Conduit GUI
    • No parameters
    • Just the order parameter
    • Just the ended parameter
    • userPHIDs, ended, and order parameters
    • objectPHIDs, ended, and order parameters
    • All parameters
  • Ran query calls from server
    • No parameters
    • Just the order parameter
    • Just the ended parameter
    • userPHIDs, ended, and order parameters
    • objectPHIDs, ended, and order parameters
    • All parameters

Diff Detail

Repository
rP Phabricator
Branch
PhrequentAPI
Lint
Lint Passed
Unit
No Test Coverage
Build Status
Buildable 674
Build 674: [Placeholder Plan] Wait for 30 Seconds

Event Timeline

emiljt retitled this revision from to Created Conduit API for Phrequent.
emiljt updated this object.
emiljt edited the test plan for this revision. (Show Details)
emiljt added a reviewer: epriestley.

@epriestley, is this code review going to be accepted? Or do you know the status of it? Is there something that I can do to help it get accepted? This is something that I was looking at extending myself but then I found this code review.

If there is anything I can do to help this change just let me know! Thanks!

Generally, Phrequent is very near the bottom of our priority list. I haven't had time to look at this, and don't expect to anytime soon.

epriestley edited edge metadata.

This should wait for T7715 and be rebuilt on top of it.

This revision now requires changes to proceed.May 13 2015, 2:03 PM