Page MenuHomePhabricator
Feed Advanced Search

Mar 25 2017

andypuettmann added a comment to Q571: How can I include any JS code in Phabricator.

Tried numerous ways to do so.
Always failed with it.

Mar 25 2017, 1:39 AM
andypuettmann updated the answer details for Q571: How can I include any JS code in Phabricator (Answer 533).
Mar 25 2017, 1:39 AM
andypuettmann added Q571: How can I include any JS code in Phabricator (Answer 533).
Mar 25 2017, 1:38 AM
chad added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

But if you say "Haskell" you may have to wait a minute.

Mar 25 2017, 1:27 AM · Conduit, Feature Request
epriestley closed T12449: Cannot set access control for creating new Maniphest forms as Resolved.

I'm going to presume this is approximately resolved.

Mar 25 2017, 1:09 AM · Bug Report
mbenua-mparticle added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

Raw HTTP is actually perfect for me! Thank you!

Mar 25 2017, 12:50 AM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

But if you say "Haskell" you may have to wait a minute.

Mar 25 2017, 12:46 AM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

Maybe easier would be for you to tell me what language you want to use and I'll just write you a snippet which encodes a request?

Mar 25 2017, 12:45 AM · Conduit, Feature Request
epriestley edited the content of Changelog.
Mar 25 2017, 12:42 AM
epriestley created 2017 Week 12 (Very Late March).
Mar 25 2017, 12:41 AM
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

The page implies that all I should need to do is POST up, for example, the following:

Mar 25 2017, 12:38 AM · Conduit, Feature Request
mbenua-mparticle added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

To be clear though, I still haven't figured out how to actually put in a POST body. It's fine to use URL params for something like differential.querydiffs, but I can imagine the problems I'll run into with URL length when doing harbormaster.sendmessage (especially when reporting a bunch of unit test and linting results).

Mar 25 2017, 12:15 AM · Conduit, Feature Request

Mar 24 2017

chad edited the content of Developer Setup.
Mar 24 2017, 10:54 PM
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

(In the longer term, the permissions here could probably be cleaned up, but I think that's the cleanest fix without upstream changes.)

Mar 24 2017, 10:53 PM · Bug Report
chad edited the content of Developer Setup.
Mar 24 2017, 10:51 PM
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

You can edit the "Create Form" form and change "Visible To:" to "Members of Project: Doctorate in Form Theory". Then only members of that project will be able to create other forms or edit the form form, I believe.

Mar 24 2017, 10:50 PM · Bug Report
alexmv added a comment to T12449: Cannot set access control for creating new Maniphest forms.

Enabling that form seems to have done the trick -- thanks! Since it got renamed at some point from "Create form," its existence was almost certainly confusing to some previous admin, which is what got us into this.

Mar 24 2017, 10:46 PM · Bug Report
epriestley added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Ah. I think you'd just see this, then:

Mar 24 2017, 10:01 PM · Owners, Audit, Restricted Project, Differential
jmeador added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

FWIW, All packages in our monorepo have weak dominion.

Mar 24 2017, 9:56 PM · Owners, Audit, Restricted Project, Differential
epriestley added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

Overall, you only get checkboxes for users, packages and projects which are already reviewers, with the exception that you'll get one for yourself if you aren't a reviewer yet.

Mar 24 2017, 9:47 PM · Owners, Audit, Restricted Project, Differential
chad renamed Equipment Purchasing from Equipment to Equipment Purchasing.
Mar 24 2017, 9:45 PM
chad edited the content of Phacility Employee Handbook.
Mar 24 2017, 9:44 PM
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

I think there is also zero legitimate reason for the form-configuring-form to be editable and this entire workflow essentially represents a trap for the unwary.

Mar 24 2017, 9:43 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

If you can't enable that form, you might need to go hunt it down in the database and manually enable it (or tell me what you hit and we'll fix it if it's a bug).

Mar 24 2017, 9:38 PM · Bug Report
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

Can you click that link, then "Enable Form"? Or do you get some kind of error?

Mar 24 2017, 9:36 PM · Bug Report
leonardorccarvalho added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Works perfectly. Thanks a lot!

Mar 24 2017, 9:36 PM · Bug Report
alexmv added a comment to T12449: Cannot set access control for creating new Maniphest forms.

I do now see that, but I'm unclear how to use its form to configure the form to configure the form to configure the form from.

Screenshot 2017-03-24 14.28.46.png (405×1 px, 54 KB)

Mar 24 2017, 9:34 PM · Bug Report
chad closed T12270: Unbeta Badges as Resolved.
Mar 24 2017, 9:32 PM · Badges (v0)
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

A MARVEL OF ACADEMIC PURITY, EDITENGINE PUSHES THE BOUNDARIES OF ADVANCED FORM THEORY TO NEW DEPTHS

Mar 24 2017, 9:27 PM · Bug Report
chad created Equipment Purchasing.
Mar 24 2017, 9:27 PM
chad edited the content of Phacility Employee Handbook.
Mar 24 2017, 9:25 PM
epriestley added a comment to T12449: Cannot set access control for creating new Maniphest forms.

DO YOU HAVE ACCESS TO FORMS FOR CONFIGURING FORMS?!

Mar 24 2017, 9:25 PM · Bug Report
alexmv created T12449: Cannot set access control for creating new Maniphest forms.
Mar 24 2017, 9:22 PM · Bug Report
jmeador added a comment to T12272: Allow users to accept on behalf of packages they control a containing package for.

How do we envision these rules applying to the following situation?

Mar 24 2017, 9:12 PM · Owners, Audit, Restricted Project, Differential
epriestley added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Mar 24 2017, 9:03 PM · Badges (v0)
epriestley added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Mar 24 2017, 9:01 PM · Badges (v0)
epriestley created T12448: Rotate 2017 SSL certificate for `secure.phabricator.com`.
Mar 24 2017, 8:47 PM · Ops
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Mar 24 2017, 8:44 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Mar 24 2017, 8:28 PM · Daemons, Ops, Phacility
epriestley closed T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad) as Resolved by committing Restricted Diffusion Commit.
Mar 24 2017, 8:14 PM · Bug Report
epriestley added a revision to T11050: Allow resigning from a review you are not directly responsible for: Unknown Object (Differential Revision).
Mar 24 2017, 8:07 PM · Prioritized, Restricted Project, Differential, Feature Request
epriestley added a revision to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad): Unknown Object (Differential Revision).
Mar 24 2017, 8:07 PM · Bug Report
epriestley added a revision to T12447: Missing documentation for crafting raw Conduit API requests: Restricted Differential Revision.
Mar 24 2017, 7:37 PM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

Yeah, there's no real hint about that in the UI to suggest that the box will update, and it's easy to miss since it looks nearly identical after it updates. I'll add a piece of hint text to hopefully make it more clear.

Mar 24 2017, 7:32 PM · Conduit, Feature Request
mbenua-mparticle added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

OH, that only happens if you do the human-readable option. I didn't notice that little box at the bottom had changed amidst the giant blob of json that the call returned.

Mar 24 2017, 7:31 PM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

Oh, maybe we could make this more clear. Here's how to use the examples to figure out how to make a call:

Mar 24 2017, 7:23 PM · Conduit, Feature Request
leonardorccarvalho added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Thanks for you help and I'm sorry the steps I provided are not enough to reproduce my problem. I will gather more information and post soon.

Mar 24 2017, 7:22 PM · Bug Report
mbenua-mparticle added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

The good news though is with that hint I seem to have it working without even needing to POST: https://phabricator.myurl.com/api/differential.querydiffs?api.token=api-TOKEN&ids[0]=5

Mar 24 2017, 7:21 PM · Conduit, Feature Request
mbenua-mparticle added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

I can definitely try either of those two options, but to my point, it wasn't obvious to me from the documentation:

pasted_file (382×1 px, 36 KB)

Mar 24 2017, 7:13 PM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

(Kicking Documentation off because this would just be another tab next to PHP / cURL / arc in Conduit, not static documentation.)

Mar 24 2017, 7:06 PM · Conduit, Feature Request
epriestley edited projects for T12447: Missing documentation for crafting raw Conduit API requests, added: Conduit; removed Documentation.
Mar 24 2017, 7:06 PM · Conduit, Feature Request
epriestley added a comment to T12447: Missing documentation for crafting raw Conduit API requests.

Can you use a service like https://requestb.in with curl?

Mar 24 2017, 7:05 PM · Conduit, Feature Request
chad added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

4f9bbb63ae9b was tested on this server. very scientific.

Mar 24 2017, 7:03 PM · Bug Report
chad added a project to T12447: Missing documentation for crafting raw Conduit API requests: Documentation.
Mar 24 2017, 6:56 PM · Conduit, Feature Request
chad added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

I am not able to reproduce the bug based on the steps you provided above. Do you have more steps or rules in place that I should be following?

Mar 24 2017, 6:54 PM · Bug Report
epriestley added a comment to T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).

Does your webserver error log have a more specific stack trace on the server? I can't reproduce this locally.

Mar 24 2017, 6:53 PM · Bug Report
mbenua-mparticle created T12447: Missing documentation for crafting raw Conduit API requests.
Mar 24 2017, 6:42 PM · Conduit, Feature Request
leonardorccarvalho created T12446: Can't create new diffs using `arc diff` after upgrade (possibly related to 0ceab7d36f914f1c7ae45b733a1ab1ae7b7afbad).
Mar 24 2017, 6:41 PM · Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Yeah I can't seem to fix this ad hoc. My guess is it's never worked to allow you to skip the first tab. If I fix the error states, it then breaks existing tab panels. So we'd need to build a migration.

Mar 24 2017, 6:40 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

I just plan to fix the bug. That's the hard part of a 2-person team with thousands of open issues. Pick and choose the battles.

Mar 24 2017, 5:05 PM · Dashboards, Bug Report
chad added a comment to D3357: GitHub-style punchcards.

This feature might be the key to unlocking growth, but the design needs a little polish. Gently placing back in your queue for 2018.

Mar 24 2017, 4:39 PM
chad requested changes to D3357: GitHub-style punchcards.
Mar 24 2017, 4:26 PM
chad removed a reviewer for D3357: GitHub-style punchcards: mshang.
Mar 24 2017, 4:26 PM
epriestley closed T12395: Calendar events aren't sufficiently exhaustive during destruction as Resolved by committing Restricted Diffusion Commit.
Mar 24 2017, 4:21 PM · Calendar
epriestley closed T12422: Unable to navigate to "All Daemons" on Mobile as Resolved by committing Restricted Diffusion Commit.
Mar 24 2017, 4:20 PM · Daemons, Mobile, Bug Report
epriestley added a revision to T12395: Calendar events aren't sufficiently exhaustive during destruction: Unknown Object (Differential Revision).
Mar 24 2017, 4:14 PM · Calendar
epriestley added a revision to T12422: Unable to navigate to "All Daemons" on Mobile: Unknown Object (Differential Revision).
Mar 24 2017, 4:10 PM · Daemons, Mobile, Bug Report
epriestley closed T12441: Implement "Phabricator Stories" as Resolved.

I don't really see a pathway to turn this into a particularly useful and/or hilarious feature that has enough of a payoff to justify the legwork.

Mar 24 2017, 4:07 PM
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Mar 24 2017, 3:56 PM · Daemons, Ops, Phacility
cjrjscgallo added a comment to Q599: Is it possible to change the maniphest.points field from an int to a select? (Answer 532).

Thanks for the response. That's what I had concluded after trying to modify it, but wanted to make sure.

Mar 24 2017, 3:52 PM
chad added Q599: Is it possible to change the maniphest.points field from an int to a select? (Answer 532).
Mar 24 2017, 3:42 PM
cjrjscgallo updated Q599: Is it possible to change the maniphest.points field from an int to a select? from to Is it possible to change the maniphest.points field from an int to a select?.
Mar 24 2017, 3:41 PM
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Mar 24 2017, 3:23 PM · Daemons, Ops, Phacility
epriestley added a revision to T12298: Allow daemon pools to autoscale down to 0 processes: Unknown Object (Differential Revision).
Mar 24 2017, 3:21 PM · Daemons, Ops, Phacility
epriestley merged T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection into T11037: `bin/phd status` is misleading about local / remote daemons.
Mar 24 2017, 3:07 PM · Daemons, Restricted Project
epriestley merged task T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection into T11037: `bin/phd status` is misleading about local / remote daemons.
Mar 24 2017, 3:07 PM · Restricted Project, Bug Report
epriestley added a comment to T12412: PhutilDaemonOverseer's pidfile updates break `phd status`'s dead-daemon detection.

Actually, this is a little different from what I've been seeing locally. I'm just going to merge this into T11037, which discusses some general improvements to phd status output.

Mar 24 2017, 3:07 PM · Restricted Project, Bug Report
cspeckmim updated the language for P2032 Creating diff from python using Conduit from remarkup to python.
Mar 24 2017, 2:11 PM
cspeckmim created P2032 Creating diff from python using Conduit.
Mar 24 2017, 2:11 PM
eliaspro added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Thinking about this again, what about making this hard-coded numbered list instead a list of draggable items like it's used in Menu Configuration?

Spectacle.n15391.png (343×853 px, 14 KB)

Mar 24 2017, 2:00 PM · Dashboards, Bug Report
chad claimed T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.
Mar 24 2017, 1:36 PM · Dashboards, Bug Report
epriestley added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

Based on like 10 seconds of looking at the code that's where I'd start. 💯

Mar 24 2017, 1:35 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

amazing what sleep will accomplish

Mar 24 2017, 1:30 PM · Dashboards, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

I think i need to set the inputs with an actual key, not just [], so [$ii]

Mar 24 2017, 1:29 PM · Dashboards, Bug Report
ioeric added a comment to T12118: Differential email does not contain reviewers/subscribers information anymore.

For me, I often need to click into the link to check if I am a reviewer or a subscriber. It would be nice if I could know who are the reviewers by looking at the email.

Mar 24 2017, 10:36 AM · Restricted Project, Differential, Bug Report
epriestley claimed T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

We should still be able to handle this case, but I can take a look at what's happening since this is a nonstandard element and the fix is probably easy once I understand what's going on with the logic.

Mar 24 2017, 10:30 AM · Dashboards, Bug Report
epriestley added a comment to T12118: Differential email does not contain reviewers/subscribers information anymore.

See also T5853 (distinguish between "blocking" and "reviewing" in email) and T10574 (provide more guidance around the pathway toward a revision becoming accepted).

Mar 24 2017, 10:25 AM · Restricted Project, Differential, Bug Report
Mnkras added a comment to T11443: T-shirts with new logo.

I want

Mar 24 2017, 4:28 AM · Public & Media Relations
chad added a comment to T11443: T-shirts with new logo.

pasted_file (398×424 px, 168 KB)

Mar 24 2017, 4:17 AM · Public & Media Relations
chad closed T11443: T-shirts with new logo as Resolved.

Ordered a new batch of tshirts at great corporate expense.

Mar 24 2017, 4:17 AM · Public & Media Relations
chad claimed T11443: T-shirts with new logo.
Mar 24 2017, 3:48 AM · Public & Media Relations
chad updated subscribers of T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

@epriestley as far as I can tell, empty tokenizers do not POST an input to evaluate.

Mar 24 2017, 3:08 AM · Dashboards, Bug Report
• tomekj2ee awarded T6868: Show when users are online or offline in Conpherence a Like token.
Mar 24 2017, 12:48 AM · Conpherence
• tomekj2ee awarded T11371: Restore PHP to Glory a Burninate token.
Mar 24 2017, 12:35 AM · Public & Media Relations
chad added a member for Community: ddfisher.
Mar 24 2017, 12:28 AM
chad added a member for Community: gregprice.
Mar 24 2017, 12:26 AM
neilfitz added a project to T12118: Differential email does not contain reviewers/subscribers information anymore: Restricted Project.

Our users are also missing this, since they can't tell whether they're a blocking/lone reviewer from the e-mail.

Mar 24 2017, 12:14 AM · Restricted Project, Differential, Bug Report
alexmv awarded T12118: Differential email does not contain reviewers/subscribers information anymore a Like token.
Mar 24 2017, 12:08 AM · Restricted Project, Differential, Bug Report

Mar 23 2017

epriestley added a comment to T9501: Need a dashboard that can used by members of differing Space/Projects.

I can't think of any offhand. It might be easier to fix the dashboard/panel stuff first (i.e., inherit parent dashboard permissions) and then do this rather than the other way around, though.

Mar 23 2017, 8:35 PM · Dashboards
chad updated subscribers of T9501: Need a dashboard that can used by members of differing Space/Projects.

@epriestley any reason to not implement Spaces on Dashboards?

Mar 23 2017, 8:03 PM · Dashboards