Page MenuHomePhabricator
Feed Advanced Search

Mar 7 2017

mbishopim3 added a comment to T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.

Occasionally, even after running the re-parse command, the basics of the commit will import but the content itself will not. A phd restart will almost immediately allow the commit to then import.

Mar 7 2017, 10:18 PM · Daemons, Bug Report
chad added a revision to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead: Unknown Object (Differential Revision).
Mar 7 2017, 9:29 PM · PHUI, Bug Report
chad added a comment to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.

But other interfaces around Phabricator are not going to fair as well, just FYI.

Mar 7 2017, 9:25 PM · PHUI, Bug Report
chad added a comment to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.

Server side reduces this quite a bit:

Mar 7 2017, 9:25 PM · PHUI, Bug Report
chad added a comment to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.

We can look at the server side issue, but we already have text-overflow on this element, so it's not going to resolve the core problem.

Mar 7 2017, 9:14 PM · PHUI, Bug Report
eliaspro added a comment to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.

I agree that long project names might a problem on its own - but for now this was a compromise I had to make to compensate for the fact that projects are just "generic" containers which can't be further classified to distinguish between ACL groups/projects/tags.

Mar 7 2017, 9:13 PM · PHUI, Bug Report
chad added a comment to T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.

This is working as intended. If you choose significantly long project names, many pieces of Phabricator will naturally become less useful and more difficult for you users to use.

Mar 7 2017, 9:02 PM · PHUI, Bug Report
mbishopim3 added a comment to T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.

More info: the commit contained binary files (executables actually).

Mar 7 2017, 8:25 PM · Daemons, Bug Report
eliaspro created T12367: Cut off Project names in policy dropdown indiscernible - need `text-overflow: ellipsis` & a tooltip instead.
Mar 7 2017, 8:00 PM · PHUI, Bug Report
eliaspro updated the task description for T12366: Long Project names outgrow dropdown widget.
Mar 7 2017, 7:55 PM · PHUI, Bug Report
chad added a revision to T12366: Long Project names outgrow dropdown widget: Unknown Object (Differential Revision).
Mar 7 2017, 7:53 PM · PHUI, Bug Report
mbishopim3 added a comment to T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.

We now have one that's failing even when forcing a reparse:

Mar 7 2017, 7:44 PM · Daemons, Bug Report
eliaspro added a project to T12366: Long Project names outgrow dropdown widget: PHUI.
Mar 7 2017, 7:21 PM · PHUI, Bug Report
eliaspro created T12366: Long Project names outgrow dropdown widget.
Mar 7 2017, 7:12 PM · PHUI, Bug Report
ksmith added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

@20after4 : Ah. So by default, everyone just gets a "create task" option, which would never use any custom forms or subtypes. That does seem to resolve my concerns (although I still think it would be cool for phab to behave differently depending on which project(s) I was viewing).

Mar 7 2017, 5:10 PM · EditEngine, Maniphest, Custom Fields, Feature Request
kionopia added a watcher for Community: kionopia.
Mar 7 2017, 4:43 PM
mbishopim3 added a comment to T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.

Is there a way to recall the logs from a failing task? Is there any value in providing the logs for the successful task? Not really sure what to say here besides that it's a consistent and random issue on our repositories. When we run the trace, it then "bumps" the task somehow and things succeed, so it would be difficult to provide detail on an error.

Mar 7 2017, 4:16 PM · Daemons, Bug Report
epriestley added a comment to T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.

There isn't enough information here for us to reproduce this, or even begin.

Mar 7 2017, 4:12 PM · Daemons, Bug Report
mbishopim3 created T12365: Daemon never recovers with Git commit parsing task, but manually reparsing succeeds.
Mar 7 2017, 4:10 PM · Daemons, Bug Report
aadis created T12364: Ways to bring arc land --delete-remote for git back.
Mar 7 2017, 3:24 PM · Feature Request
eliaspro added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.
In T12363#214538, @chad wrote:

What are you attempting to do here or was this accidental?

Mar 7 2017, 3:15 PM · Dashboards, Bug Report
chad closed T12362: PhrequentSearchEngine missing in Dashboards as Invalid.
Mar 7 2017, 2:40 PM · Dashboards, Phrequent, Bug Report
chad added a comment to T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.

What are you attempting to do here or was this accidental?

Mar 7 2017, 2:39 PM · Dashboards, Bug Report
eliaspro created T12363: Configuration of "Tab Panel" is lost when 1st tab has no panel assigned.
Mar 7 2017, 11:35 AM · Dashboards, Bug Report
avivey added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

I wouldn't expect anything in the logs; You can try running a script and running something like this:

$d = new DateTimeZone("Asia/Jerusalem");
var_dump($d->getOffset(new DateTime('@'.time())));

IF you're getting back FALSE, then it's definitely this.

Mar 7 2017, 10:21 AM · Bug Report
cinek added a comment to T12362: PhrequentSearchEngine missing in Dashboards.

Hooray!

Mar 7 2017, 10:11 AM · Dashboards, Phrequent, Bug Report
20after4 added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

@ksmith: Thanks to the new 'favorites' menu, each user can customize their menu, there is no longer a global 'create new' menu, as it's been replaced.

Mar 7 2017, 9:43 AM · EditEngine, Maniphest, Custom Fields, Feature Request
fooishbar awarded T12314: Support formal task subtypes (like "bug" vs "feature") a Mountain of Wealth token.
Mar 7 2017, 9:25 AM · EditEngine, Maniphest, Custom Fields, Feature Request
johnny-bit added a comment to T12362: PhrequentSearchEngine missing in Dashboards.

This was fixed in rPb28da10336ca 😄

Mar 7 2017, 9:18 AM · Dashboards, Phrequent, Bug Report
cinek created T12362: PhrequentSearchEngine missing in Dashboards.
Mar 7 2017, 9:03 AM · Dashboards, Phrequent, Bug Report
bcorcoran added a comment to T12360: PhabricatorDataNotAttachedException on /people/commits/ID/.

Beautiful and timely fix. Thanks!

Mar 7 2017, 5:05 AM · Diffusion, People, Bug Report
bcorcoran awarded T12360: PhabricatorDataNotAttachedException on /people/commits/ID/ a Like token.
Mar 7 2017, 4:56 AM · Diffusion, People, Bug Report
chad edited the content of Developer Setup.
Mar 7 2017, 2:24 AM
chad edited the content of Developer Setup.
Mar 7 2017, 2:24 AM
chad added a project to T12356: Calendar Notification is not respecting my Time Zone Settings: Calendar.
Mar 7 2017, 2:16 AM · Calendar, Bug Report
chad closed T12361: Test as Spite.
Mar 7 2017, 1:59 AM · Feature Request
• ahn.jihyun created T12361: Test.
Mar 7 2017, 1:35 AM · Feature Request
chad closed T12360: PhabricatorDataNotAttachedException on /people/commits/ID/ as Resolved by committing Restricted Diffusion Commit.
Mar 7 2017, 1:24 AM · Diffusion, People, Bug Report
chad added a revision to T12360: PhabricatorDataNotAttachedException on /people/commits/ID/: Unknown Object (Differential Revision).
Mar 7 2017, 12:44 AM · Diffusion, People, Bug Report
chad claimed T12360: PhabricatorDataNotAttachedException on /people/commits/ID/.
Mar 7 2017, 12:43 AM · Diffusion, People, Bug Report
zhiyuchen added a comment to T12279: Timezone Asia/Jerusalem incorrectly listed under UTC-0.

@avivey Thanks! my php version is PHP 5.5.9, would you recommend upgrading? I didn't find anything in my php error log though, any suggestion where else I could look into to find out the cause?

Mar 7 2017, 12:18 AM · Bug Report

Mar 6 2017

bcorcoran created T12360: PhabricatorDataNotAttachedException on /people/commits/ID/.
Mar 6 2017, 11:28 PM · Diffusion, People, Bug Report
ksmith added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

Our phabricator instance (WMF) is shared by many(!) different projects and teams. If I understand the current proposal, all the subtypes and forms would be available to all users across all teams and projects, except where specific access is limited (e.g. security issues). I think that forces us to either: a) have very few subtypes, which manage to achieve consensus, or b) have a lot of subtypes, which would mean that everyone's "create new" menu would be very cluttered.

Mar 6 2017, 11:25 PM · EditEngine, Maniphest, Custom Fields, Feature Request
chad changed the edit policy for Cowboy Commits.
Mar 6 2017, 9:44 PM
eliaspro added a comment to T5380: Better handling of anchored links in Phriction.

I'd love to see anchor indicators when hovering headings.

Mar 6 2017, 9:38 PM · Phriction, Remarkup
johnny-bit added a comment to T12324: Dashboards that contain a restricted panel have all query panels stuck in 'Loading...'.

You guys fast.

Mar 6 2017, 9:13 PM · Dashboards, Bug Report
chad added a comment to T12324: Dashboards that contain a restricted panel have all query panels stuck in 'Loading...'.

I fixed that hours ago.

Mar 6 2017, 8:45 PM · Dashboards, Bug Report
johnny-bit added a comment to T12324: Dashboards that contain a restricted panel have all query panels stuck in 'Loading...'.

Hi! I think I've hit similar issue, but for another type of panel. We've used to use PhrequentSearchEngine panel to show phrequent/query/tracking/ (Currently tracking), which apparently got removed in D17341. Currently tracking on main dashboard was quite interesting indicator for current work aaand could be fun until phrequent gets more useul 😄

Mar 6 2017, 8:37 PM · Dashboards, Bug Report
epriestley closed T12359: Registration does not accept valid email addresses as Wontfix.

You can adjust the validation here:

Mar 6 2017, 8:24 PM · Bug Report
lambdafu added a comment to T3179: Importing data from external systems (like GitHub, JIRA, Trac, Asana, Trello, etc.).

I have written a couple of scripts to semi-automatically convert trac ("done") and roundup (WIP) to phabricator. The included patch based on D10754 but extended does quite a good job at faking timestamps even for subtasks and notifications etc, but the whole thing is neither complete nor bullet-proof or even convenient. Good enough for people who don't mind to get their hands dirty: https://github.com/lambdafu/phabricator-test

Mar 6 2017, 8:22 PM · Guides, Doorkeeper
lambdafu updated the task description for T12359: Registration does not accept valid email addresses.
Mar 6 2017, 8:17 PM · Bug Report
lambdafu created T12359: Registration does not accept valid email addresses.
Mar 6 2017, 8:13 PM · Bug Report
alexmv updated the task description for T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.
Mar 6 2017, 7:23 PM · Installing & Upgrading, Differential
chad updated the task description for T12270: Unbeta Badges.
Mar 6 2017, 7:02 PM · Badges (v0)
chad added a revision to T12270: Unbeta Badges: Unknown Object (Differential Revision).
Mar 6 2017, 6:08 PM · Badges (v0)
chad added a comment to T12357: Remarkup: **bold** effect in Header section is not discernible.

I went with a different color, but we could also do text-shadow or a background-color... lmk if this seems bad in practice.

Mar 6 2017, 5:37 PM · Remarkup, Bug Report
chad closed T12357: Remarkup: **bold** effect in Header section is not discernible as Resolved by committing Restricted Diffusion Commit.
Mar 6 2017, 5:28 PM · Remarkup, Bug Report
epriestley added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

I think that's right, yes.

Mar 6 2017, 5:28 PM · Installing & Upgrading, Differential
epriestley added a comment to T12337: Provide conduit access to related tasks and in particular 'mentions'.

Users do count, but I thiiiiiink user mentions are a special kind of mention because they have more complicated rules and you can't read them with edge.search today. T4654 discusses better rules around "notify me when I'm mentioned"; there are some related Conpherence-specific tasks too.

Mar 6 2017, 5:23 PM · Feature Request
epriestley closed T12358: Not secure generating as Invalid.

Sometimes, Apache is configured with a rule which prevents any access to any URL which contains /..

Mar 6 2017, 5:15 PM · Bug Report
chad added a revision to T12357: Remarkup: **bold** effect in Header section is not discernible: Unknown Object (Differential Revision).
Mar 6 2017, 3:58 PM · Remarkup, Bug Report
chad added a comment to T12357: Remarkup: **bold** effect in Header section is not discernible.
In T12357#214422, @chad wrote:

What would you suggest as a fix?

Mar 6 2017, 3:31 PM · Remarkup, Bug Report
chad added a comment to T12357: Remarkup: **bold** effect in Header section is not discernible.

What would you suggest as a fix?

Mar 6 2017, 3:26 PM · Remarkup, Bug Report
chad added a comment to T12358: Not secure generating.

This is not a bug. If you feel this is a security issue, please file an appropriate task on HackerOne. https://hackerone.com/phabricator

Mar 6 2017, 3:17 PM · Bug Report
d.mulyavko created T12358: Not secure generating.
Mar 6 2017, 3:12 PM · Bug Report
avivey added a comment to T12356: Calendar Notification is not respecting my Time Zone Settings.

I would argue that at least for email ("External Targets"), we should always include the timezone when rendering times. When we allowed custom time-format, I've also added it there (Partly because there wasn't the time-zone warning we have now, partly because I kept my phabricator config to PST when traveling, and partly to make copying the time easier).

Mar 6 2017, 10:38 AM · Calendar, Bug Report
avivey renamed T12357: Remarkup: **bold** effect in Header section is not discernible from Display bold text with a different color in titles to Remarkup: **bold** effect in Header section is not discernible.
Mar 6 2017, 10:31 AM · Remarkup, Bug Report
mcorteel created T12357: Remarkup: **bold** effect in Header section is not discernible.
Mar 6 2017, 10:23 AM · Remarkup, Bug Report
borna.emami added Q117: Is there a way to bulk accept audits? (Answer 523).
Mar 6 2017, 10:11 AM
alexmv added a comment to T12085: Upgrading: Changes to Differential Custom Fields in Commit Messages.

The table in the description references a getConduitFieldKey method – which I can't find reference to ever existing in the source. Was that supposed to be getFieldKeyForConduit?

Mar 6 2017, 7:15 AM · Installing & Upgrading, Differential
friesenschrauber created T12356: Calendar Notification is not respecting my Time Zone Settings.
Mar 6 2017, 6:58 AM · Calendar, Bug Report
20after4 added a comment to T12314: Support formal task subtypes (like "bug" vs "feature").

Weakness: Subtyping is Probably Only Useful in Maniphest

Subtyping is likely to involve a large number of changes in shared infrastructure (EditEngine), but they will probably only ever be useful in Maniphest. Although most applications now use EditEngine, I can't really come up with any good use cases for subtyping in other applications. Perhaps Calendar could use subtyping on events, but this feels like a solution searching for a problem.

Mar 6 2017, 12:44 AM · EditEngine, Maniphest, Custom Fields, Feature Request
20after4 added a comment to T12337: Provide conduit access to related tasks and in particular 'mentions'.

I don't suppose users count as objects in this context? I'd love to have a way to be notified of new @mentions for my username. A client app that queries conduit occasionally and sends me desktop notifications, well, that would just be super cool.

Mar 6 2017, 12:33 AM · Feature Request
20after4 awarded T12337: Provide conduit access to related tasks and in particular 'mentions' a Mountain of Wealth token.
Mar 6 2017, 12:29 AM · Feature Request
20after4 added a comment to T12353: Convert PhabricatorSearchDocumentQuery to cursor based paging.

I agree that results should almost always be on the first page and ideally near the top of the first page. I'm also tempted to say that 10 results per page might be closer to the right number than 100.

Mar 6 2017, 12:28 AM · Elasticsearch, Wikimedia, Search

Mar 5 2017

eliaspro added a comment to T9363: improve handling of panels that do not always return results.

While the number (>0) of query results would decide whether a given panel is shown or not, also exposing the actual number of results would be IMHO interesting in the context of tabbed panels, so that each tab title could be shown as demonstrated in the mock-up below (please excuse my non-designer-ish CSS skills):

Mar 5 2017, 10:46 PM · Dashboards (v2), ApplicationSearch, Restricted Project, Restricted Project
eliaspro added a comment to T6532: Support "Login As" functionality for Administrators.

I agree with the general understanding of Adminstrators not being in absolute power.
But then again, it would be useful to be able to have a way to test the effective policies applied to a given user.

Mar 5 2017, 9:54 PM
eliaspro added a comment to T12355: Daemons are having a rough time starting on `admin`.

In terms of application/code stability, it would be good to handle this properly.

Mar 5 2017, 9:02 PM · Phacility, Daemons, Ops
eadler closed D17468: Remove the the duplicate words.
Mar 5 2017, 6:37 PM
epriestley accepted D17468: Remove the the duplicate words.

courageous 🎆

Mar 5 2017, 6:12 PM
eadler created D17468: Remove the the duplicate words.
Mar 5 2017, 5:56 PM
chad updated subscribers of T10319: Generate unique? profile images for each new user.

True story, I built these because of @jmeador

Mar 5 2017, 5:05 PM · Feature Request, People
epriestley added a comment to T10319: Generate unique? profile images for each new user.

I dropped the cache, looks like they're working.

Mar 5 2017, 5:01 PM · Feature Request, People
chad added a comment to T10319: Generate unique? profile images for each new user.

I presume these will pop in as cache expires?

Mar 5 2017, 4:29 PM · Feature Request, People
chad added a comment to T10319: Generate unique? profile images for each new user.

rolling.

Mar 5 2017, 4:26 PM · Feature Request, People
chad closed T10319: Generate unique? profile images for each new user as Resolved by committing Restricted Diffusion Commit.
Mar 5 2017, 4:25 PM · Feature Request, People
epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

Specifically, I suspect it won't affect repo because db restarts separately, so the database won't be unavailable when daemons on repo restart.

Mar 5 2017, 12:32 PM · Phacility, Daemons, Ops
epriestley added a comment to T12355: Daemons are having a rough time starting on `admin`.

It looks like this might be an issue with MySQL not being available yet when the daemons start. The deployment script restarts MySQL, then immediately restarts the daemons, and a bunch of this stuff is ending up in the log in the 1-2 seconds after the restart:

Mar 5 2017, 12:32 PM · Phacility, Daemons, Ops
epriestley added a project to T12355: Daemons are having a rough time starting on `admin`: Phacility.
Mar 5 2017, 12:24 PM · Phacility, Daemons, Ops
epriestley created T12355: Daemons are having a rough time starting on `admin`.
Mar 5 2017, 12:24 PM · Phacility, Daemons, Ops
epriestley added a comment to T12353: Convert PhabricatorSearchDocumentQuery to cursor based paging.

I believe that's unavoidable with offset-based paging (the solution is to switch to cursor-based paging, but I think we can't do that for fulltext).

Mar 5 2017, 12:18 PM · Elasticsearch, Wikimedia, Search
20after4 added a comment to T12353: Convert PhabricatorSearchDocumentQuery to cursor based paging.

I'll investigate further into the slowness (I wasn't involved in the debugging so far so I don't know more than what's said in the task)

Mar 5 2017, 11:22 AM · Elasticsearch, Wikimedia, Search
filterfish added a comment to T12349: The language stanza in a code block shouldn't be indexed..

Hrrm, it looks like my reply to your email bounced. Short answer is that we don't offer anything special for academic or open source right now. We'd like to in the future, but are focusing on building our core business first.

Mar 5 2017, 8:43 AM · Bug Report
filterfish added a comment to T12349: The language stanza in a code block shouldn't be indexed..

The document contains "bash". This behavior seems correct to me.

Mar 5 2017, 8:40 AM · Bug Report
constantx edited the content of Organizations Using Phabricator.
Mar 5 2017, 7:35 AM
epriestley edited the content of 2017 Week 9 (Early March).
Mar 5 2017, 2:34 AM
joshuaspence edited the content of 2017 Week 9 (Early March).
Mar 5 2017, 2:24 AM
chad added a revision to T10319: Generate unique? profile images for each new user: Unknown Object (Differential Revision).
Mar 5 2017, 12:36 AM · Feature Request, People
epriestley added a comment to T10319: Generate unique? profile images for each new user.

🍞 🐦

Mar 5 2017, 12:30 AM · Feature Request, People
epriestley added a comment to T10319: Generate unique? profile images for each new user.

fancy 👑

Mar 5 2017, 12:30 AM · Feature Request, People