Page MenuHomePhabricator
Feed All Stories

Feb 15 2017

chad updated the task description for T12270: Unbeta Badges.
Feb 15 2017, 9:14 PM · Badges (v0)
epriestley added a project to T12272: Allow users to accept on behalf of packages they control a containing package for: Owners.
Feb 15 2017, 9:02 PM · Owners, Audit, Restricted Project, Differential
epriestley created T12272: Allow users to accept on behalf of packages they control a containing package for.
Feb 15 2017, 9:02 PM · Owners, Audit, Restricted Project, Differential
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Feb 15 2017, 9:00 PM · Badges (v0)
chad added a comment to T12270: Unbeta Badges.

I'm just trying to understand the feedback. There are points that come very close to "redesign badges" to which I'm like years away from doing, if ever.

Feb 15 2017, 8:37 PM · Badges (v0)
epriestley added a comment to T12270: Unbeta Badges.

I'm fine with just declining to implement that, it just seems like something we should fix if we're calling it a bug.

Feb 15 2017, 8:36 PM · Badges (v0)
epriestley created T12271: When users accept a revision, allow them to accept for a subset of package/project reviewers they could accept for.
Feb 15 2017, 8:32 PM · Audit, Restricted Project, Differential
chad added a comment to T12270: Unbeta Badges.

Why do we need T10698 specifically? Seems overkill.

Feb 15 2017, 8:24 PM · Badges (v0)
ftdysa backed I2 Donate to Phacility with $25.00 USD.
Feb 15 2017, 8:23 PM · Phacility
chad created T12270: Unbeta Badges.
Feb 15 2017, 8:23 PM · Badges (v0)
chad closed T12269: New badge transaction says "created this object". as Resolved.

Darn race condition.

Feb 15 2017, 7:59 PM · Badges (v0)
chad added a comment to T11432: Ability to exclude self ("Current Viewer") from Feed panels.

pasted_file (218×242 px, 13 KB)

Feb 15 2017, 7:12 PM · Feed, Feature Request
nornagon added a comment to T11432: Ability to exclude self ("Current Viewer") from Feed panels.

Yep, this is no longer a problem with the new audit bucketing. 🏅

Feb 15 2017, 7:11 PM · Feed, Feature Request
chad closed T10473: Dashboard panel tab transactions are illegible as Resolved by committing Restricted Diffusion Commit.
Feb 15 2017, 7:05 PM · Dashboards (v2), Bug Report
chad added a revision to T10473: Dashboard panel tab transactions are illegible: Unknown Object (Differential Revision).
Feb 15 2017, 6:46 PM · Dashboards (v2), Bug Report
chad edited this Badge.
Feb 15 2017, 6:38 PM
jcox backed I2 Donate to Phacility with $20.00 USD.
Feb 15 2017, 6:37 PM · Phacility
chad removed a revision from T12269: New badge transaction says "created this object".: Unknown Object (Differential Revision).
Feb 15 2017, 6:03 PM · Badges (v0)
chad added a revision to T12269: New badge transaction says "created this object".: Unknown Object (Differential Revision).
Feb 15 2017, 6:03 PM · Badges (v0)
chad added a revision to T10798: Limit, order recipients list in BadgeView: Unknown Object (Differential Revision).
Feb 15 2017, 5:54 PM · Badges (v0), Contributor Onboarding, Restricted Project
chad created T12269: New badge transaction says "created this object"..
Feb 15 2017, 5:54 PM · Badges (v0)
avivey added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

Yeah, that looks like it's ignoring the encoding, and considers the whole argument as a parameter name.

Feb 15 2017, 4:24 PM
OCram added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

I think there is an issue with appliction/json data. If I Provide the data with a valid api.token Conduit returns:

pasted_file (338×585 px, 13 KB)

Feb 15 2017, 2:42 PM
avivey added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

I think there's a way to send raw json using curl, but if you go to the conduit app and invoke it, you get an example at the bottom of how to repeat the request using arc or curl.
It looks like curl is using some http-query-magic way to encode the request. You can maybe try -d <raw json> with -H 'Content-Type: application/json' instead, it might also work.

Feb 15 2017, 12:14 PM
OCram added a comment to Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).

Well, I have to use curl. Is there a difference in the way how conduit parses the args?

Feb 15 2017, 11:28 AM
epriestley created T12268: Make the grey "Unverified User" dot accurately reflect the new universal behavior.
Feb 15 2017, 11:19 AM · Mail, People
epriestley added a comment to T12266: arc set-config encoding XXX not work.

I think it was historically possible for third-party code to read custom config, but not possible for them to add new settings until D16928 (Nov 2016), which might be why we don't warn about bad values. Should be fine to do so now, though.

Feb 15 2017, 11:13 AM · Bug Report
epriestley edited this Badge.
Feb 15 2017, 11:07 AM
avivey updated the answer details for Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).
Feb 15 2017, 10:38 AM
avivey added Q567: JSON Example for Conduit method maniphest.edit? (Answer 510).
Feb 15 2017, 10:38 AM
avivey added a comment to T12266: arc set-config encoding XXX not work.

Well, it looks like we didn't expect users to set encoding from arc:

Feb 15 2017, 10:11 AM · Bug Report
avivey updated the task description for T12266: arc set-config encoding XXX not work.
Feb 15 2017, 9:49 AM · Bug Report
avivey closed T12267: ArcanistLintEngine::getLineAndCharFromOffset(string, integer) returns the wrong values as Invalid.

Closing, because it's probably not a bug. Comment again if you think it is. But if you do, a smaller file would help debugging.

Feb 15 2017, 9:48 AM · Bug Report
tgr added a comment to T4828: Suggest/propose possible duplicates when creating a new task.

Wikimedia ran a vote to find which problems are the most annoying to developers (not limited to problems with Phabricator) and this one was the most voted by far, with over one third of the participants voting on it.

Feb 15 2017, 9:27 AM · Maniphest, Wikimedia
OCram updated Q567: JSON Example for Conduit method maniphest.edit? from to JSON Example for Conduit method maniphest.edit?.
Feb 15 2017, 9:23 AM · Conduit
fosskevin backed I1 Test Initiative with $77.00 USD.
Feb 15 2017, 9:11 AM · Fund, Phortune
iana added a comment to T12267: ArcanistLintEngine::getLineAndCharFromOffset(string, integer) returns the wrong values.

Actually I think this is my fault, I was providing the offset in Unicode characters, not in bytes.

Feb 15 2017, 8:09 AM · Bug Report
iana created T12267: ArcanistLintEngine::getLineAndCharFromOffset(string, integer) returns the wrong values.
Feb 15 2017, 7:54 AM · Bug Report
xian0617 created T12266: arc set-config encoding XXX not work.
Feb 15 2017, 7:14 AM · Bug Report
chad merged T12265: Allow viewing passwords to password members in Passphrase into T4721: Improve user-facing documentation for Passphrase.
Feb 15 2017, 6:56 AM · Documentation, Passphrase
chad merged task T12265: Allow viewing passwords to password members in Passphrase into T4721: Improve user-facing documentation for Passphrase.
Feb 15 2017, 6:56 AM · Passphrase, Bug Report
saggid updated the task description for T12265: Allow viewing passwords to password members in Passphrase.
Feb 15 2017, 5:32 AM · Passphrase, Bug Report
saggid created T12265: Allow viewing passwords to password members in Passphrase.
Feb 15 2017, 5:31 AM · Passphrase, Bug Report
chad added a comment to T10671: Implement `badges.search` and `badges.edit` Conduit API methods.

Looks like search and edit are implemented, but not awards. Awarding a badge through conduit seems nice to have? but I don't see any requests for it, so not sure it's worth holding up unbeta.

Feb 15 2017, 5:31 AM · Restricted Project, Conduit, Badges (v0)
hach-que added a comment to T7149: Allow users to import data into a new Phacility instance.

I think we're going to end up trialling it with one project for now (just in case there's some prototype stuff that we're using on the self-hosted instance that we run into on Phacility), and then probably manually move other things over if we decide to pull the switch (just because we'll have done work on both at that point).

Feb 15 2017, 4:58 AM · Phacility
chad claimed T10798: Limit, order recipients list in BadgeView.
Feb 15 2017, 4:15 AM · Badges (v0), Contributor Onboarding, Restricted Project
chad moved T10798: Limit, order recipients list in BadgeView from Backlog to v0 on the Badges board.
Feb 15 2017, 4:15 AM · Badges (v0), Contributor Onboarding, Restricted Project
chad moved T9006: Allow users to pick which Badges show on Timeline from v1 to Backlog on the Badges board.
Feb 15 2017, 4:15 AM · Badges, Restricted Project
chad closed T6526: Implement "Badges" or "Medals", for annotating special users as Resolved.

This has been implemented and in use for a while, so closing and will finish up smaller tasks in backlog.

Feb 15 2017, 4:14 AM · Restricted Project, Badges (v0), People
chad claimed T10671: Implement `badges.search` and `badges.edit` Conduit API methods.
Feb 15 2017, 4:14 AM · Restricted Project, Conduit, Badges (v0)
chad closed T11167: Unbeta Feedback on Badges as Resolved.

punt

Feb 15 2017, 4:12 AM · Restricted Project, Badges (v0)
chad added a comment to T7149: Allow users to import data into a new Phacility instance.

Just need to tarball it up and upload it to us, and we'll do the import when your ready.

Feb 15 2017, 3:56 AM · Phacility
hach-que added a comment to T7924: Unprototype Calendar (v1).

Is there any way of getting Calendar enabled on a Phacility instance? We're looking to migrate from a self-hosted instance, and it's one of the applications we use to organise meetings, etc.

Feb 15 2017, 3:51 AM · Restricted Project, Prioritized, Calendar
hach-que added a comment to T7149: Allow users to import data into a new Phacility instance.

Is there an import process available for customers of the Phacility free tier? We have a reasonably complex, self-hosted set up, and we're looking at Phacility but currently have no easy way of migrating the data.

Feb 15 2017, 2:58 AM · Phacility
chad triaged T8033: Inform editor when panels have more restrictive policy than dashboard as Normal priority.
Feb 15 2017, 2:14 AM · Dashboards
chad closed T12207: Sort out the great Dashboard column editing mess as Resolved.
Feb 15 2017, 2:14 AM · Dashboards (v2)
chad added a comment to T12207: Sort out the great Dashboard column editing mess.

Gonna just close this since I don't think the issues were really that great and I got most of the major bugs this week. I'll pick away at some of the others.

Feb 15 2017, 2:14 AM · Dashboards (v2)
epriestley closed T11114: Move Differential to EditEngine as Resolved.

I think we've ended up in a fairly stable place here, and remaining work generally has existing tasks motivating it in terms of features (like T10967 for reviewers, T10574 for acceptance rules, T2543 for revision states, various tasks for inlines, T9713 for commit message parsing). Obsoleted transaction types also need migrations (but they should happen alongside the migrations that other features require so we're minimizing the number of weeks with migrations that need special planning) and this dumb mess got left in PhabricatorModularTransaction.php and will probably be there for an uncomfortably long time:

Feb 15 2017, 1:23 AM · Prioritized, Restricted Project, Infrastructure, EditEngine, Differential
chad added a comment to T12263: Javascript licence.

externally linked?

Feb 15 2017, 12:13 AM · Feature Request

Feb 14 2017

terirem added a comment to T12263: Javascript licence.

So all javascripts (including externally linked) are under Apache 2.0?

Feb 14 2017, 11:53 PM · Feature Request
eadler closed T12176: Importing large SVN repository causes a full disk as Invalid.

closed per user request

Feb 14 2017, 11:03 PM · Bug Report (Needs Information), Daemons, Diffusion
matthewrbowker added a comment to T12176: Importing large SVN repository causes a full disk.

@epriestley Please close this task (I can't).

Feb 14 2017, 10:54 PM · Bug Report (Needs Information), Daemons, Diffusion
chad created T12264: Install Dashboard to Home / Favorites / Project.
Feb 14 2017, 10:28 PM · Profile Menu Item, Dashboards (v2)
chad removed a project from T12136: What to do with "Application Counts" with MenuItem: Dashboards (v2).
Feb 14 2017, 10:16 PM · Profile Menu Item
chad closed T12248: You can drag out a tab panel for some reason in dashboard arrange, a subtask of T12207: Sort out the great Dashboard column editing mess, as Resolved.
Feb 14 2017, 10:16 PM · Dashboards (v2)
chad closed T12248: You can drag out a tab panel for some reason in dashboard arrange as Resolved by committing Restricted Diffusion Commit.
Feb 14 2017, 10:16 PM · Dashboards (v2)
chad added a revision to T12248: You can drag out a tab panel for some reason in dashboard arrange: Unknown Object (Differential Revision).
Feb 14 2017, 10:11 PM · Dashboards (v2)
chad closed T11449: Selecting tabs for a Dashboard tab panel uses a plain <select /> which is a big mess as Resolved by committing Restricted Diffusion Commit.
Feb 14 2017, 9:43 PM · Dashboards (v2), Feature Request
remusvrm updated the question details for Q566: How to separate developer tickets from client tickets?.
Feb 14 2017, 9:36 PM
remusvrm updated Q566: How to separate developer tickets from client tickets? from to How to separate developer tickets from client tickets?.
Feb 14 2017, 9:36 PM
chad added a revision to T11449: Selecting tabs for a Dashboard tab panel uses a plain <select /> which is a big mess: Unknown Object (Differential Revision).
Feb 14 2017, 9:34 PM · Dashboards (v2), Feature Request
remusvrm closed Q565: Is it possible to change the limit of the listed tasks per page in Query? as resolved.
Feb 14 2017, 9:09 PM
remusvrm added a comment to Q565: Is it possible to change the limit of the listed tasks per page in Query?.

Ohh, that is embarrassing... Now I found where to the limit can be changed. Thanks for the help.

Feb 14 2017, 9:09 PM
remusvrm added a comment to Q564: How to remove an image from a task?.

ooh, sorry. I believed that the linked task is an example. Ok, there is a ticket for it. Subscribed.

Feb 14 2017, 9:06 PM · Maniphest
chad added a comment to T10608: Arc diff failing to generate diff with svn 1.9.3.

See Planning.

Feb 14 2017, 9:00 PM · Arcanist, Bug Report
fmcat added a comment to T10608: Arc diff failing to generate diff with svn 1.9.3.

I am getting the same error with

arcanist c58f1b9a2507488b2152473dd1f0bbc7e99c09c1 (29 Apr 2016)
libphutil 0709cd5cfc2616c29104387de6dde3de1db7fa32 (2 May 2016)

and

svn, version 1.9.4 (r1740329)
   compiled Apr 28 2016, 01:15:57 on x86_64-apple-darwin15.4.0
Feb 14 2017, 8:49 PM · Arcanist, Bug Report
chad added a comment to Q564: How to remove an image from a task?.

Did you try reading the task I linked above?

Feb 14 2017, 8:48 PM · Maniphest
chad added a comment to Q565: Is it possible to change the limit of the listed tasks per page in Query?.

Did you try clicking the button that says "Edit Query"

Feb 14 2017, 8:47 PM
remusvrm added a comment to Q565: Is it possible to change the limit of the listed tasks per page in Query?.

For example https://secure.phabricator.com/maniphest/query/open . It lists only 100. How can I list all the tickets?
I checked the config of Manifest, but I didn't find anything related with the listed tasks.

Feb 14 2017, 8:41 PM
remusvrm added a comment to Q564: How to remove an image from a task?.

But I don't want a new photo. I want to remove the photo.

Feb 14 2017, 8:39 PM · Maniphest
chad added a comment to Q564: How to remove an image from a task?.

Drag a different photo on top of it.

Feb 14 2017, 8:35 PM · Maniphest
remusvrm added a comment to Q564: How to remove an image from a task?.

Yes, from this site. If you click here Abuse and search for task T7593, you see there is an image attached to the ticket in the Workboard. If you edit the ticket, you don't find something like {Fxxxx}.

Feb 14 2017, 8:29 PM · Maniphest
chad added a comment to Q565: Is it possible to change the limit of the listed tasks per page in Query?.

Where have you looked for this field that you are not able to find it?

Feb 14 2017, 7:31 PM
chad claimed T11449: Selecting tabs for a Dashboard tab panel uses a plain <select /> which is a big mess.
Feb 14 2017, 7:02 PM · Dashboards (v2), Feature Request
epriestley added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

We'll probably implement a Phabricator-side mute-this-thread in connection with T10448, I'm just guessing that some of the users complaining that GMail mute doesn't work won't consider clicking a link in the email body, then clicking "Mute Thread" to be an acceptable solution.

Feb 14 2017, 7:01 PM · Restricted Project, Mail, Feature Request
remusvrm updated Q565: Is it possible to change the limit of the listed tasks per page in Query? from to Is it possible to change the limit of the listed tasks per page in Query?.
Feb 14 2017, 7:00 PM
jmeador added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

We could also implement "Mute Thread" in Phabricator, but I assume no one would be willing to click twice to mute a thread.

Feb 14 2017, 6:57 PM · Restricted Project, Mail, Feature Request
epriestley added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

I'm pretty sure we can't generate our own Message-ID with Amazon SES, at least -- although we use the HTTP API, not the SMTP API, and maybe the rules differ. Not sure about other providers.

Feb 14 2017, 6:56 PM · Restricted Project, Mail, Feature Request
eadler merged T12250: Add repository short name as a variable in build steps into T10517: Make Harbormaster build variables extensible.
Feb 14 2017, 6:56 PM · Harbormaster, Restricted Project, Feature Request
eadler merged task T12250: Add repository short name as a variable in build steps into T10517: Make Harbormaster build variables extensible.
Feb 14 2017, 6:56 PM · Harbormaster, Feature Request
eadler added a comment to T12240: Best of Both Worlds: one email to everyone with privacy controls and good threading.

(dropped this since I don't have time to fully test, but I might come back to this in the future)

Feb 14 2017, 6:53 PM · Restricted Project, Mail, Feature Request
epriestley closed T12263: Javascript licence as Invalid.

See the LICENSE file for license information.

Feb 14 2017, 6:49 PM · Feature Request
terirem created T12263: Javascript licence.
Feb 14 2017, 6:39 PM · Feature Request
chad added a comment to Q564: How to remove an image from a task?.

Is this question about a specific task on this site, secure.phabricator.com, or some other site? Your question does not contain enough details to give you an answer.

Feb 14 2017, 5:19 PM · Maniphest
remusvrm updated Q564: How to remove an image from a task? from to How to remove an image from a task?.
Feb 14 2017, 4:57 PM · Maniphest
NLDK updated Q563: Maniphest Export Points from to Maniphest Export Points.
Feb 14 2017, 3:49 PM · Maniphest
farrago added a comment to T8298: Fix Windows Escaping.

Just a quick note to say that I have been using @BYK 's D15675, D15676 and D15693 for over a month now with no issues, so would be in favour of them being merged in.
Setup:

  • Windows 10 x64,
  • standalone cmd.exe
  • cmd.exe through ConEmu
  • git repos hosteded locally in phabricator (running on Ubuntu server)
  • relatively straightforward use of arc feature, arc diff and arc land
Feb 14 2017, 3:27 PM · Windows
epriestley edited the content of Starmap.
Feb 14 2017, 3:25 PM
chad closed T10145: Adding a panel to a dashboard multiple times, a subtask of T12207: Sort out the great Dashboard column editing mess, as Resolved.
Feb 14 2017, 3:24 PM · Dashboards (v2)
chad closed T10145: Adding a panel to a dashboard multiple times as Resolved by committing Restricted Diffusion Commit.
Feb 14 2017, 3:24 PM · Dashboards (v2), Bug Report