Page MenuHomePhabricator
Feed All Stories

Nov 16 2016

epriestley reopened T8588: One instance has a severe, global performance issue as "Open".
Nov 16 2016, 11:55 PM · Restricted Project
epriestley reopened T8588: One instance has a severe, global performance issue, a subtask of T7664: Moving a task on a workboard is very slow (and sometimes times out with a fatal error), as Open.
Nov 16 2016, 11:55 PM · Restricted Project, Workboards
epriestley reopened T8588: One instance has a severe, global performance issue, a subtask of T8444: Increase typeahead speed, as Open.
Nov 16 2016, 11:55 PM · Typeahead, PHUI, Restricted Project
michaeljs1990 placed T11517: arc lint missed some error reported from the golint up for grabs.
Nov 16 2016, 11:08 PM · Bug Report (Needs Information), Arcanist, Lint
• Gadget0624 awarded T2374: Have `bin/phd stop` look for daemons and warn you about them, or nuke them with some flag a Hungry Hippo token.
Nov 16 2016, 9:08 PM · OS Students 2013, Daemons
chad added a comment to T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.

I'll generate a best guess patch later today.

Nov 16 2016, 6:50 PM · Bug Report, Quicksand, Conpherence (v4)
OCram added a comment to Q518: Have there been any changes in the way phabricator shows file attachements? (Answer 477).

What kind of information could be important here? I do my best to help...

Nov 16 2016, 6:48 PM
chad added a comment to T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.

I think it's a race condition with quicksand

Nov 16 2016, 6:46 PM · Bug Report, Quicksand, Conpherence (v4)
avivey changed the visibility for T2374: Have `bin/phd stop` look for daemons and warn you about them, or nuke them with some flag.
Nov 16 2016, 6:44 PM · OS Students 2013, Daemons
OCram added a comment to T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.

@flippingtables: Do you access to your phabrictor intance through a loadbalancer?

Nov 16 2016, 6:43 PM · Bug Report, Quicksand, Conpherence (v4)
chad renamed T11785: Clicking on images for lightbox won't load with Persistent Chat enabled from Clicking on a media does not trigger preview, refreshing the page now loads url of media to Clicking on images for lightbox won't load with Persistent Chat enabled.
Nov 16 2016, 6:33 PM · Bug Report, Quicksand, Conpherence (v4)
OCram awarded T11785: Clicking on images for lightbox won't load with Persistent Chat enabled a Heartbreak token.
Nov 16 2016, 6:33 PM · Bug Report, Quicksand, Conpherence (v4)
OCram added a comment to T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.

Ran into this too.

Nov 16 2016, 6:33 PM · Bug Report, Quicksand, Conpherence (v4)
OCram added a comment to T11883: Remarkup embedded images doesnt load.

You are right, I also can't reproduce it on the Phacility instance. I really don't know what do do next ^^

Nov 16 2016, 6:31 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
chad added a comment to T11883: Remarkup embedded images doesnt load.

Please take the time to try to reproduce the issue on a Phacility instance, and report back if you can. We have not been able to reproduce the issue with these set up steps. Reproduction steps must includes everything we need to review and fix the issue locally.

Nov 16 2016, 6:28 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
OCram updated the task description for T11883: Remarkup embedded images doesnt load.
Nov 16 2016, 6:27 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
chad merged T11883: Remarkup embedded images doesnt load into T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.
Nov 16 2016, 6:25 PM · Bug Report, Quicksand, Conpherence (v4)
chad merged task T11883: Remarkup embedded images doesnt load into T11785: Clicking on images for lightbox won't load with Persistent Chat enabled.
Nov 16 2016, 6:25 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
avivey added a project to T11883: Remarkup embedded images doesnt load: Quicksand.
Nov 16 2016, 6:23 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
chad claimed T11884: Comment text areas lost the green highlight when a file is dropped into them in the recent comment update.
Nov 16 2016, 6:21 PM · Remarkup, Design
csilvers added a comment to T11882: Suggestion for easier port handling when setting up phabricator as a git repo.

Fair enough.

Nov 16 2016, 6:21 PM · Feature Request
epriestley created T11884: Comment text areas lost the green highlight when a file is dropped into them in the recent comment update.
Nov 16 2016, 6:19 PM · Remarkup, Design
epriestley added a comment to T11882: Suggestion for easier port handling when setting up phabricator as a git repo.

I made the claim that this is the easiest way because my experience is that "have all your users do something" is always harder than "do something yourself", at least in the roles I held at Facebook and here. I'd gladly solder pins on the motherboard if I didn't have to tell users to make a simple configuration change.

Nov 16 2016, 6:17 PM · Feature Request
OCram created T11883: Remarkup embedded images doesnt load.
Nov 16 2016, 6:11 PM · Quicksand, Remarkup, Files, Conpherence, Bug Report
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

Ah. I believe I fixed a simpler version of that case earlier, but not the specific case you describe. D16881 should fix that case.

Nov 16 2016, 6:08 PM · Prioritized, Transactions
chad added a comment to Q518: Have there been any changes in the way phabricator shows file attachements? (Answer 477).

We'd love to track this down as 2-3 installs have this problem with persistent chat, but no one has been able to reproduce the issue here. I've tried lots of different ways. It would be great to have a reproduction case.

Nov 16 2016, 6:05 PM
epriestley added a revision to T7643: Improve prose diffs (was: description changes don't generate usable diffs): Unknown Object (Differential Revision).
Nov 16 2016, 6:05 PM · Prioritized, Transactions
OCram closed Q518: Have there been any changes in the way phabricator shows file attachements? as resolved.
Nov 16 2016, 6:01 PM · Files
OCram added Q518: Have there been any changes in the way phabricator shows file attachements? (Answer 477).
Nov 16 2016, 6:01 PM
csilvers created T11882: Suggestion for easier port handling when setting up phabricator as a git repo.
Nov 16 2016, 5:58 PM · Feature Request
dseifert added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

@epriestley One issue that is still open is adding a change to the beginning and to the end of a large prose text. The diff of that becomes unusable. This is still reproducable with the current stable branch )(fc71a7e92dc26e0d93ec33d709391411d2bcd827 (Mon, Nov 14)). For details / reproduction steps please see T11825.

Nov 16 2016, 5:34 PM · Prioritized, Transactions
epriestley closed T7643: Improve prose diffs (was: description changes don't generate usable diffs) as Resolved.
  • D16853 tweaked one issue that I caught.
  • T11881 is a followup for restoring code diffs in a small set of scenarios.
  • I haven't seen other issues for a while, so I'm going to consider this resolved until they crop up. If you do run into cases where the new algorithm isn't doing a great job, let us know.
Nov 16 2016, 5:17 PM · Prioritized, Transactions
epriestley created T11881: Make Paste content changes render as code diffs again, not prose diffs.
Nov 16 2016, 5:15 PM · Restricted Project, Paste, Transactions
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.
  • D16877 includes the entire CLDR "windows to the rest of the world" map in things we'll recognize.
    • Neither that map or the Microsoft map have "Russian Time", but maybe that's just what the UI shows for zones like "Russia Time Zone 10", and not what actually appears in the ICS file? Otherwise we're going to have to keep digging.
  • D16878 fixes the form customization stuff I mentioned above, mostly by preventing quirky/confusing fields from being customized (at least, for now).
  • D16879 fixes a rendering bug with "All Day" events that could shift them forward or backwards by a day.
  • D16880 uses the same rendering logic for subheaders (which now appears to be correct) and tooltips (which had a different, older copy of very similar logic).
Nov 16 2016, 3:53 PM · Restricted Project, Calendar
epriestley added a comment to T8644: Herald template / recipe book / examples / quick-create tool.

From T11878, these patterns aren't obvious:

Nov 16 2016, 3:48 PM · Restricted Project, Herald
epriestley added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

It also occurs to me that if I needed multiple and conditions I could make a rule that did nothing but just matched those conditions,

Nov 16 2016, 3:43 PM · Feature Request
ofbeaton added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

Thank you very much for your reply @epriestley, and I see what you were trying to say now, I wasn't understanding you before. You were right, I can indeed use your solution (with some modifications), I had not noticed the any of change compared to my own rules.

Nov 16 2016, 3:41 PM · Feature Request
epriestley added a revision to T11816: Calendar: Feedback and Suggestions: Unknown Object (Differential Revision).
Nov 16 2016, 3:37 PM · Restricted Project, Calendar
epriestley added a revision to T11816: Calendar: Feedback and Suggestions: Unknown Object (Differential Revision).
Nov 16 2016, 3:24 PM · Restricted Project, Calendar
epriestley added a revision to T11816: Calendar: Feedback and Suggestions: Unknown Object (Differential Revision).
Nov 16 2016, 2:59 PM · Restricted Project, Calendar
epriestley added a revision to T11816: Calendar: Feedback and Suggestions: Unknown Object (Differential Revision).
Nov 16 2016, 2:48 PM · Restricted Project, Calendar
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.
  • Editing the EditEngine default form and removing "Host" means that "Host" no longer prefills. This is the same as T10222. A temporary workaround would be to prevent that field from being defaulted. It should be defaultable in the long run, but is probably low-value to assign a default to, so it's probably better to just lock it for now and make it behave a little more sensibly.
Nov 16 2016, 2:17 PM · Restricted Project, Calendar
epriestley added a comment to T11816: Calendar: Feedback and Suggestions.

Some other stuff:

Nov 16 2016, 2:15 PM · Restricted Project, Calendar
epriestley added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

epriestley's "Another herald rule" condition does not solve the problem.

Nov 16 2016, 2:04 PM · Feature Request
ofbeaton added a comment to T8783: Unprototype Nuance (v1).

Relevant to my feature request in T11878, if Nuance could also work on incoming Tasks, that would make that feature request redundant as I wouldn't need to use Herald rules instead.

Nov 16 2016, 1:55 PM · Nuance, Restricted Project
ofbeaton added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

OK, I'll make my feature request there next time. I didn't ask for support, I made a feature request. I thought those were separate things.

Nov 16 2016, 1:51 PM · Feature Request
epriestley added a comment to T7789: Support Git Large File Storage.

No, there's no specific support for LFS mirroring yet.

Nov 16 2016, 1:13 PM · Customer Impact, Restricted Project, Files, Diffusion
epriestley closed T11879: Highlight trailing white spaces at end of lines as Resolved.

Configure ArcanistTextLinter, which can detect and automatically correct these errors:

Nov 16 2016, 1:01 PM · Feature Request
epriestley added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

Oh, sorry! I added you to Community so you should now have edit power.

Nov 16 2016, 12:58 PM · Projects (v3), Feature Request
epriestley added a member for Community: aklapper.
Nov 16 2016, 12:57 PM
ioeric created T11879: Highlight trailing white spaces at end of lines.
Nov 16 2016, 12:41 PM · Feature Request
babic.boris awarded T5427: Force a line break in a table cell a Like token.
Nov 16 2016, 12:07 PM · Remarkup
Unknown Object (User) added a comment to T7789: Support Git Large File Storage.

This doesn't add any compatibility with mirroring, does it?

Nov 16 2016, 10:24 AM · Customer Impact, Restricted Project, Files, Diffusion
aklapper added a comment to T10308: Bring back links to task lists on project pages; List name of assignee in workboard cards.

Could someone with superpowers please edit the task summary to describe the actual task content? Like
Bring back links to task lists on project pages; list name of assignee in workboard cards
? (Or who do I have to bribe to get such rights?) Thanks in advance.

Nov 16 2016, 10:22 AM · Projects (v3), Feature Request
OCram updated Q518: Have there been any changes in the way phabricator shows file attachements? from to Have there been any changes in the way phabricator shows file attachements?.
Nov 16 2016, 9:08 AM · Files
• hamipl added a comment to T11850: Pholio Mockups Image Uploads stops uploading.

http2 apache Mod caused this bug.
I disabled http2 mod and everythings fine now

Nov 16 2016, 7:14 AM · Bug Report (Needs Information)
jasonrumney added a comment to T11816: Calendar: Feedback and Suggestions.

It's my fault - I edited the comment to add that link (probably based on
the same search you did) in parallel with your reply.

Nov 16 2016, 4:56 AM · Restricted Project, Calendar
chad added a comment to T11816: Calendar: Feedback and Suggestions.

Whoops... just read your full comment. Sorry for dupe.

Nov 16 2016, 4:53 AM · Restricted Project, Calendar
chad added a comment to T11816: Calendar: Feedback and Suggestions.

Is this the complete list?

Nov 16 2016, 4:52 AM · Restricted Project, Calendar
jasonrumney added a comment to T11816: Calendar: Feedback and Suggestions.

Unfortunately "W. Europe Standard Time" is just one of the many unnecessarily verbose timezone names that Microsoft uses. I also had "Singapore Standard Time" and "Russian Time" (which I presume refers to Moscow, rather than any of the other dozen or so timezones in Russia) in my calendar for this week.

Nov 16 2016, 4:46 AM · Restricted Project, Calendar
0 awarded T11816: Calendar: Feedback and Suggestions a Love token.
Nov 16 2016, 4:31 AM · Restricted Project, Calendar
0 added a comment to T11816: Calendar: Feedback and Suggestions.
  • D16866 is just what I needed, thanks!
  • D16867 fixes what I reported, but I hadn't noticed that the month view tooltips are also affected in the same way:
    All day event tooltip (239×642 px, 10 KB)
Nov 16 2016, 4:31 AM · Restricted Project, Calendar
cspeckmim edited the content of Community Resources.
Nov 16 2016, 3:23 AM
chad added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

I'd prefer you ask these questions in Ponder or Conpherence. It's not our expectation to provide this level of support to the community for free.

Nov 16 2016, 3:09 AM · Feature Request
ofbeaton added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

I really hope I'm not missing something here... sincere apologies if I am...

Nov 16 2016, 2:44 AM · Feature Request
dmgarcia closed Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? as resolved.
Nov 16 2016, 2:22 AM · Mail
dmgarcia added a comment to Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 475).

Ok thanks :)

Nov 16 2016, 2:21 AM
chad added a comment to Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 475).

Two way communication with a task in Phabricator with a non-user is Nuance, we don't have support for what you want today.

Nov 16 2016, 2:20 AM
chad updated the answer details for Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 475).
Nov 16 2016, 2:19 AM
dmgarcia updated the answer details for Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 476).
Nov 16 2016, 2:17 AM
chad added a comment to Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 476).

Click "Add Comment" to my answer if you have more questions please.

Nov 16 2016, 2:17 AM
dmgarcia added a comment to Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 475).

Thanks @chad, but the ticket creation is working, just the next interactions that's not.

Nov 16 2016, 2:17 AM
dmgarcia added Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 476).
Nov 16 2016, 2:16 AM
chad added Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? (Answer 475).
Nov 16 2016, 2:12 AM
dmgarcia updated Q517: Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"? from to Is there any configuration to allow outbound e-mails to unregistered users aka "Email user"?.
Nov 16 2016, 1:59 AM · Mail
epriestley added a revision to T11044: Support partitioning application databases across multiple database hosts: Unknown Object (Differential Revision).
Nov 16 2016, 1:17 AM · Clusters, Prioritized
epriestley added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

Why doesn't this work?

Nov 16 2016, 12:31 AM · Feature Request
chad added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

You can also write your own HeraldAdapter.

Nov 16 2016, 12:30 AM · Feature Request
ofbeaton added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

Unfortunately the Another Herald Rule condition does not do what I want, if I understand it correctly.

Nov 16 2016, 12:26 AM · Feature Request

Nov 15 2016

epriestley added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

You can already do this with the "Another Herald Rule" condition.

Nov 15 2016, 11:35 PM · Feature Request
chad added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:27 PM · Feature Request
chad added a comment to T11878: New Herald Action To trigger evaluation of another herald rule.

Sorry, I merged this the wrong way. Nuance is our planned "routing engine" aka Help Desk. We plan to pursue it first and it should be the tool you use to evaluate incoming new objects and where they go.

Nov 15 2016, 11:27 PM · Feature Request
chad merged T11878: New Herald Action To trigger evaluation of another herald rule into T8783: Unprototype Nuance (v1).
Nov 15 2016, 11:26 PM · Nuance, Restricted Project
chad merged task T11878: New Herald Action To trigger evaluation of another herald rule into T8783: Unprototype Nuance (v1).
Nov 15 2016, 11:26 PM · Feature Request
chad reopened T8783: Unprototype Nuance (v1) as "Open".
Nov 15 2016, 11:25 PM · Nuance, Restricted Project
chad reopened T8783: Unprototype Nuance (v1), a subtask of T5815: Make it easier to work through a queue of stuff, as Open.
Nov 15 2016, 11:25 PM · Restricted Project, Maniphest, Differential, Audit
chad merged T8783: Unprototype Nuance (v1) into T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:23 PM · Feature Request
chad merged task T8783: Unprototype Nuance (v1) into T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:23 PM · Nuance, Restricted Project
ofbeaton updated the task description for T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:16 PM · Feature Request
ofbeaton updated the task description for T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:15 PM · Feature Request
ofbeaton updated the task description for T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:13 PM · Feature Request
ofbeaton updated the task description for T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:12 PM · Feature Request
ofbeaton created T11878: New Herald Action To trigger evaluation of another herald rule.
Nov 15 2016, 11:11 PM · Feature Request
chad added a comment to T11877: closing a subtask as a duplicate of another task doesn't update parent task.

Or we already have a task on it. Either way.

Nov 15 2016, 10:48 PM · Bug Report
chad added a comment to T11877: closing a subtask as a duplicate of another task doesn't update parent task.

This is expected, we merge no information over when you close a task as a duplicate.

Nov 15 2016, 10:48 PM · Bug Report
epriestley merged task T11877: closing a subtask as a duplicate of another task doesn't update parent task into T3287: Merging duplicate tasks doesn't migrate dependencies.
Nov 15 2016, 10:40 PM · Bug Report
epriestley merged T11877: closing a subtask as a duplicate of another task doesn't update parent task into T3287: Merging duplicate tasks doesn't migrate dependencies.
Nov 15 2016, 10:40 PM · Maniphest, Hard
justinkao created T11877: closing a subtask as a duplicate of another task doesn't update parent task.
Nov 15 2016, 10:25 PM · Bug Report
avivey added a comment to T3626: Detection of language for some file types isn't good.

In a similar approach, this code just makes pygmentize spit out it's data in a nicer format:

Nov 15 2016, 10:23 PM · Customer Impact, Restricted Project, Differential
epriestley created E1212: Running Errands.
Nov 15 2016, 9:47 PM