Page MenuHomePhabricator
Feed All Stories

Jul 7 2016

epriestley added a revision to T9275: Move Calendar to EditEngine: Unknown Object (Differential Revision).
Jul 7 2016, 5:26 PM · ApplicationEditor, Policy, Calendar
eadler moved T10628: Merge "Local Commits" and "Open Revisions Affecting These Files" into tabs on "Revision Update History" in Differential from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jul 7 2016, 5:21 PM · Restricted Project, Differential
eadler added a project to T10393: Differential revision's images cannot be viewed as raw files: Restricted Project.
Jul 7 2016, 5:21 PM · Restricted Project, Differential, Files, Bug Report
eadler added a project to T10628: Merge "Local Commits" and "Open Revisions Affecting These Files" into tabs on "Revision Update History" in Differential: Restricted Project.
Jul 7 2016, 5:20 PM · Restricted Project, Differential
eadler added a project to T11218: Differential advanced search: bucket/group by repository: Restricted Project.
Jul 7 2016, 5:19 PM · Restricted Project, Differential, Feature Request
eadler added a project to T10725: Can't access binary files of differential revision: Restricted Project.
Jul 7 2016, 5:19 PM · Restricted Project, Differential, Bug Report
eadler added a project to T10856: Small Revision page enhanchments: Restricted Project.
Jul 7 2016, 5:18 PM · Restricted Project, Feature Request, Differential
eadler added a project to T11050: Allow resigning from a review you are not directly responsible for: Restricted Project.
Jul 7 2016, 5:17 PM · Prioritized, Restricted Project, Differential, Feature Request
eadler added a project to T11093: DifferentialInlineCommentEditor retains state across Quicksand requests: Restricted Project.
Jul 7 2016, 5:16 PM · Inline Comments, Restricted Project, Quicksand, Differential
eadler added a subtask for T1460: Add "Done" link to an inline comment: T8130: Keyboard shortcut for "Done".
Jul 7 2016, 5:16 PM · User Delight, Differential
eadler added a parent task for T8130: Keyboard shortcut for "Done": T1460: Add "Done" link to an inline comment.
Jul 7 2016, 5:16 PM · Inline Comments, Restricted Project, Differential
eadler added a project to T8130: Keyboard shortcut for "Done": Restricted Project.
Jul 7 2016, 5:16 PM · Inline Comments, Restricted Project, Differential
eadler added a project to T8163: Show changes since last accepted version: Restricted Project.
Jul 7 2016, 5:15 PM · Restricted Project, Differential
eadler added a subtask for T1460: Add "Done" link to an inline comment: T8250: Would like to see all comments and their status in a list.
Jul 7 2016, 5:15 PM · User Delight, Differential
eadler added a parent task for T8250: Would like to see all comments and their status in a list: T1460: Add "Done" link to an inline comment.
Jul 7 2016, 5:15 PM · Inline Comments, Prioritized, Restricted Project, Differential
eadler added a project to T8250: Would like to see all comments and their status in a list: Restricted Project.
Jul 7 2016, 5:14 PM · Inline Comments, Prioritized, Restricted Project, Differential
eadler added a project to T9947: "Expand File" keyboard shortcut in Differential should work on files which were not loaded by default: Restricted Project.
Jul 7 2016, 5:14 PM · Restricted Project, Differential, Feature Request
eadler added a project to T8420: Hidden inlines don't work properly with anchor-based navigation: Restricted Project.
Jul 7 2016, 5:14 PM · Inline Comments, Restricted Project, Differential
eadler added a project to T8573: All reviewers should be allowed to mark a comment as done: Restricted Project.
Jul 7 2016, 5:14 PM · Restricted Project, Differential, Restricted Project
eadler added a project to T8838: Interdiffs of unchanged files claim to be empty.: Restricted Project.
Jul 7 2016, 5:13 PM · Restricted Project, Differential
eadler added a project to T8900: Copying contents from Phriction history diff doesn't work: Restricted Project.
Jul 7 2016, 5:11 PM · Restricted Project, Infrastructure, Differential, Phriction
eadler added a project to T1751: Consider ways to improve revert handling: Restricted Project.
Jul 7 2016, 5:10 PM · Restricted Project, Diffusion, Differential
eadler added a project to T7870: List of cases where ghost comments end up somewhere confusing or unexpected: Restricted Project.
Jul 7 2016, 5:10 PM · Inline Comments, Restricted Project, Differential
eadler added a project to T7773: Allow code reviews across multiple repos: Restricted Project.
Jul 7 2016, 5:10 PM · Restricted Project, Differential
eadler added a project to T7739: Redesign Lint/Unit properties : Restricted Project.
Jul 7 2016, 5:10 PM · Restricted Project, Harbormaster, Differential
eadler added a project to T3960: Add capability to hide lint messages when viewing a diff: Restricted Project.
Jul 7 2016, 5:09 PM · Restricted Project, Differential
eadler added a project to T5853: Hint to blocking reviewers that they're blocking in earlier acceptance emails: Restricted Project.
Jul 7 2016, 5:00 PM · Restricted Project, Differential
eadler added a project to T6273: Inline comments do not support editing (Pholio, Differential, Diffusion): Restricted Project.
Jul 7 2016, 5:00 PM · Audit, Restricted Project, Diffusion, Differential, Transactions, Pholio
eadler added a project to T7467: Consider giving deleted files a more prominent treatment in the changeset UI: Restricted Project.
Jul 7 2016, 4:59 PM · Restricted Project, Differential
eadler added a subtask for T4768: Phabricator silently overwrites concurrent changes (no conflict detection): T7657: Reviewer changes across `arc diff --update` and web UI are not merged constructively.
Jul 7 2016, 4:59 PM · Restricted Project, ApplicationEditor, Wikimedia, Maniphest, Phriction
eadler added a parent task for T7657: Reviewer changes across `arc diff --update` and web UI are not merged constructively: T4768: Phabricator silently overwrites concurrent changes (no conflict detection).
Jul 7 2016, 4:58 PM · Restricted Project, Differential, Arcanist
eadler added a comment to T7657: Reviewer changes across `arc diff --update` and web UI are not merged constructively.

Ref T4768

Jul 7 2016, 4:58 PM · Restricted Project, Differential, Arcanist
eadler added a project to T7657: Reviewer changes across `arc diff --update` and web UI are not merged constructively: Restricted Project.
Jul 7 2016, 4:57 PM · Restricted Project, Differential, Arcanist
eadler added a project to T7662: "View Unified" ignores "Show Entire File" option.: Restricted Project.
Jul 7 2016, 4:54 PM · Restricted Project, Differential
eadler added a project to T7704: Rejecting then accepting a revision with blocking reviewers doesn't correctly change state: Restricted Project.
Jul 7 2016, 4:54 PM · Restricted Project, Differential
epriestley added a comment to T11286: make it possible to remove tasks from a batch edit tasks workflow.

Workaround is to edit this URI:

Jul 7 2016, 4:39 PM · Maniphest, Feature Request
eadler created T11286: make it possible to remove tasks from a batch edit tasks workflow.
Jul 7 2016, 4:35 PM · Maniphest, Feature Request
rencris updated the task description for T11285: PhabricatorApplicationTransactionPublishWorker fails on bad getURI call.
Jul 7 2016, 4:25 PM · Bug Report
rencris added a comment to T11285: PhabricatorApplicationTransactionPublishWorker fails on bad getURI call.

I've looked at the database, specifically the worker_activetask table. The bad transactions appear to be linked to profile panel objects (objectPHID like PHID-PANL-...).

Jul 7 2016, 4:22 PM · Bug Report
rencris created T11285: PhabricatorApplicationTransactionPublishWorker fails on bad getURI call.
Jul 7 2016, 3:41 PM · Bug Report
• vanblaze added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

You are right, it does not specifically describe or refer to an import button anywhere.

Jul 7 2016, 3:28 PM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Okay, but the docs don't actually describe or refer to an import button anywhere -- there's no text like "click the import button" that I missed when updating things?

Jul 7 2016, 2:58 PM · Design, Diffusion
• vanblaze added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Another issue is that when following the user docs, the docs still points towards having the import button on the creation of project.

I can't find where we say this in the documentation. Can you point me at what you're seeing?

Jul 7 2016, 2:44 PM · Design, Diffusion
chad added a comment to T10350: Scripts to migrate old "points" fields and move projects beneath other projects.

@RuralHunter these aren't versions of Phabricator we produce. How have you modified your local install?

Jul 7 2016, 1:59 PM · Projects (v3)
epriestley added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

so the UI should still look good given that edge-case.

Jul 7 2016, 1:56 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
chad added a comment to T9640: Make Phabricator compatible with PHP7.

At HEAD you should receive a setup warning that PHP7 is not supported. We don't recommend using it for production.

Jul 7 2016, 1:55 PM · Prioritized, Restricted Project, Infrastructure
chad added a comment to T11269: Repository update command fails on empty Git repositories.

Why do you have local changes in your phabricator folder?

Jul 7 2016, 1:52 PM · Diffusion, Bug Report
chad added Q440: Merging two Phabricator instances (Answer 416).
Jul 7 2016, 1:48 PM
fanis added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

Although the new graph display has the branch-like display at the left, that column is removed from the task titles (2 columns away) - which aren't indented to clearly show the parent-child relationship.

Jul 7 2016, 1:39 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
MaXxXo updated Q440: Merging two Phabricator instances from to Merging two Phabricator instances.
Jul 7 2016, 1:38 PM
sgielen added a comment to T8850: Add a setup warning to make sure users haven't wiped out $_SERVER['REMOTE_ADDR'].

I just had this problem as well. Turns out I had a typo in preamble.php. A more descriptive error would be nice, but with this task open at least the cause is easy to find :)

Jul 7 2016, 12:30 PM · Contributor Onboarding, Setup
sgielen added a comment to T10794: Upgrading: Changes to Aphlict (Real-Time Notifications) Configuration.

Note that it is possible to set a path to the websocket for every server as well. This is handy, for example, when your notification websocket is hosted behind the same URL as Phabricator itself. We have https://phabricator.ourcompany.net/ for Phabricator itself, and https://phabricator.ourcompany.net/ws/ for the notifictions websocket. This corresponds to the following client config:

Jul 7 2016, 12:00 PM · Installing & Upgrading, Aphlict
dtf added a comment to T11284: Allow line breaks in list items.

Maybe something like long lists with "titles" or "summaries" for list elements?

# Something top-level
  Long, multi-sentence description of this top-level list item.
 - Sub-element with "title" <br/>
   Long, multi-sentence description of this inner list item.
 - **Another sub-element, highlighted even** <br/>
   Another long description.
   Very long and very descriptive. 
# Something else

which renders like:

1. Something top-level
   Long, multi-sentence description of this top-level list item.
Jul 7 2016, 9:03 AM · Remarkup
dtf added a comment to T11284: Allow line breaks in list items.
Jul 7 2016, 9:01 AM · Remarkup
RuralHunter added a comment to T10350: Scripts to migrate old "points" fields and move projects beneath other projects.

It seems the move_beneath.php doesn't work any more with the latest code.

Jul 7 2016, 8:36 AM · Projects (v3)
• minishine added a comment to T11269: Repository update command fails on empty Git repositories.

Sorry to trouble you @epriestley , I have the same question(T11273) . After your committing rP5ffdb732735c, I tried to update my phabricator under the direction of doc(https://secure.phabricator.com/book/phabricator/article/upgrading/). While I executed git pull,the terminal shows

屏幕快照 2016-07-07 15.45.45.png (262×1 px, 61 KB)
. So should I commit local changes or checkout them?

Jul 7 2016, 7:49 AM · Diffusion, Bug Report
• khanhicetea added a comment to T9640: Make Phabricator compatible with PHP7.

So what exactly do you need to comment out for this to work on PHP7? Is is src/applications/cache/spec/PhabricatorDataCacheSpec.php and src/applications/cache/spec/PhabricatorOpcodeCacheSpec.php

Jul 7 2016, 7:14 AM · Prioritized, Restricted Project, Infrastructure
rbalik added a comment to T8821: arc commands (diff, version) fails on Windows - terrible fix inside.

I just ran into the issue with this error on Windows:
Usage Exception: arc diff` is only supported under git, hg, svn.`

Jul 7 2016, 4:20 AM · Windows, libphutil, Arcanist
chad closed T9360: Unbeta Phame as Resolved.
Jul 7 2016, 1:58 AM · Phame
epriestley added a revision to T9895: Update Phame Documentation: Unknown Object (Differential Revision).
Jul 7 2016, 1:40 AM · Phame
epriestley closed T9895: Update Phame Documentation as Resolved.

Resolved by D16246, I suppose.

Jul 7 2016, 1:40 AM · Phame
epriestley closed T9895: Update Phame Documentation, a subtask of T9360: Unbeta Phame, as Resolved.
Jul 7 2016, 1:40 AM · Phame
epriestley moved T6299: Remarkup links to objects in Phame break if a CNAME is set from Backlog to Future on the Phame board.
Jul 7 2016, 1:40 AM · Haskell.org, Remarkup, Phame
epriestley removed a subtask for T9897: Unbeta Feedback on Phame: T6299: Remarkup links to objects in Phame break if a CNAME is set.
Jul 7 2016, 1:40 AM · Phame
epriestley removed a parent task for T6299: Remarkup links to objects in Phame break if a CNAME is set: T9897: Unbeta Feedback on Phame.
Jul 7 2016, 1:40 AM · Haskell.org, Remarkup, Phame
epriestley added a revision to T9360: Unbeta Phame: Unknown Object (Differential Revision).
Jul 7 2016, 1:34 AM · Phame
avivey added a comment to T10349: Projects v3 Errata.

Users are somewhat surprised that adding a project removes another project (Due to being sub/parent-project); Partly that's because there's no hint in the editor that this is about to happen, or that the 2 projects are related.

Jul 7 2016, 1:23 AM · Restricted Project, Projects (v3)
chad added a revision to T9360: Unbeta Phame: Unknown Object (Differential Revision).
Jul 7 2016, 1:14 AM · Phame
epriestley added a comment to T9360: Unbeta Phame.

Oh, now that we gave posts the J monogram, we should probably also find a way to put that on the blog page somewhere (at least the internal one, and maybe the ApplicationSearch results page). Right now you have to know that you can fish the ID out of the URI. Let me see if I can poke at that a bit with my doc stuff.

Jul 7 2016, 1:02 AM · Phame
cspeckmim added a comment to T9360: Unbeta Phame.

I've been pretty excited about this since the unbetaing started.

Jul 7 2016, 12:35 AM · Phame
chad updated the task description for T9360: Unbeta Phame.
Jul 7 2016, 12:27 AM · Phame
chad added a comment to T9360: Unbeta Phame.

I think I started the unbeta task in December. So... Merry Christmas!

Jul 7 2016, 12:26 AM · Phame
epriestley added a comment to T9360: Unbeta Phame.

Just the T6299 stuff, I think.

Jul 7 2016, 12:26 AM · Phame
chad added a comment to T9360: Unbeta Phame.

Anything left to fix before unbeta?

Jul 7 2016, 12:23 AM · Phame
chad added a comment to T11284: Allow line breaks in list items.

Cursory research yields Markdown uses two spaces to indicate a <br />.

Jul 7 2016, 12:04 AM · Remarkup

Jul 6 2016

CodeMouse92 added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

@epriestley, the use case is actually from my content development department. It's hard to explain the task, but the hundreds of direct subtasks was unavoidable, and the project workboard did not work well in their workflow. Developing content for educational software can be extremely demanding. That said, I'll admit it isn't common to have hundreds of subtasks. Still, it is a scenario that can happen, so the UI should still look good given that edge-case.

Jul 6 2016, 11:57 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
epriestley added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

I'm leaning toward making it a "Relationships" box with "Task Graph", "Mocks" (with a thumbnail/gallery view instead of just titles), and "Commits/Revisions" (probably with a little more status information than we currently show).

Jul 6 2016, 11:50 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Another issue is that when following the user docs, the docs still points towards having the import button on the creation of project.

Jul 6 2016, 11:31 PM · Design, Diffusion
epriestley added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

I'm leaning toward just adding a big "Clippy" element to this UI:

Jul 6 2016, 11:30 PM · Design, Diffusion
cspeckmim added a comment to T10907: Older project's icon changes back after a few days.

ghostbusters.jpg (393×640 px, 182 KB)

Jul 6 2016, 11:09 PM · Projects, Bug Report (Needs Information)
isfs added a comment to T10907: Older project's icon changes back after a few days.
  • Choose the default colour+icon picture for the project
  • In the project history, click the link to the file that has been assigned as the picture
  • Note that the file is temporary, so will be cleaned up by the garbage collector
Jul 6 2016, 11:07 PM · Projects, Bug Report (Needs Information)
isfs added a revision to T10907: Older project's icon changes back after a few days: Unknown Object (Differential Revision).
Jul 6 2016, 11:06 PM · Projects, Bug Report (Needs Information)
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

I think if you haven't activated the repository and visit URIs, we can show an "Import" type note and call it a day for now.

Jul 6 2016, 10:55 PM · Design, Diffusion
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Yeah, I think it goes away after you push "activate"

Jul 6 2016, 10:54 PM · Design, Diffusion
cspeckmim added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Is that text already present and I just missed it?

Jul 6 2016, 10:48 PM · Design, Diffusion
chad added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

We can probably start with a text hint on this view:

Jul 6 2016, 10:44 PM · Design, Diffusion
• vanblaze added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

Another issue is that when following the user docs, the docs still points towards having the import button on the creation of project. I was pretty confused as it clearly wasn't that. Perhaps the page after clicking the 'Create Repository' could have an option of importing, which presets the values to observe a repo.

Jul 6 2016, 10:37 PM · Design, Diffusion
cspeckmim added a comment to T11278: It's confusing to figure out in the UI how to import a repository now.

I was confused at first - from previous workflow I thought I took a wrong step when setting up the repository and landed on a "completed" repository page which I didn't expect without selecting host vs. mirror. I deleted the repository and started again and finally got it after hours of turmoil (maybe more like a minute of prodding).

Jul 6 2016, 10:22 PM · Design, Diffusion
chad added a comment to T11284: Allow line breaks in list items.

Can you give an example of what specifically you are trying to build in Phabricator?

Jul 6 2016, 10:12 PM · Remarkup
Krinkle updated the task description for T11284: Allow line breaks in list items.
Jul 6 2016, 9:42 PM · Remarkup
Krinkle created T11284: Allow line breaks in list items.
Jul 6 2016, 9:33 PM · Remarkup
Luke081515.2 added an inline comment to M1458: Homepage 2016.

A few questions...

Jul 6 2016, 8:51 PM
chad added a revision to T9360: Unbeta Phame: Unknown Object (Differential Revision).
Jul 6 2016, 8:46 PM · Phame
chad added a revision to T9360: Unbeta Phame: Unknown Object (Differential Revision).
Jul 6 2016, 8:33 PM · Phame
johnny-bit added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

That would be perfect!

Jul 6 2016, 8:25 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
chad added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

It was merged into master today.

Jul 6 2016, 7:33 PM · Daemons, Bug Report
chad updated the task description for T9360: Unbeta Phame.
Jul 6 2016, 7:32 PM · Phame
ftdysa added a comment to T11270: PHP 7 does not reasonably support asynchronous signal handling until `async_signals` lands.

Looks like this should be available in 7.1 btw: https://wiki.php.net/rfc/async_signals#proposed_voting_choices.

Jul 6 2016, 7:12 PM · Daemons, Bug Report
chad added a comment to T4788: Allow "Edit Dependencies" both ways (blocking and depending).

One possible solution might be to make "Task Graph" box "Task Dependecies" and have a separate tab for a straight list (default, parent/sibling only) and graph.

Jul 6 2016, 6:55 PM · Prioritized, FreeBSD, User Delight, Wikimedia, Maniphest
chad merged task T11283: Make Task Graph Collapsable into T4788: Allow "Edit Dependencies" both ways (blocking and depending).
Jul 6 2016, 6:28 PM · Maniphest, Bug Report