Page MenuHomePhabricator
Feed All Stories

Apr 12 2017

epriestley merged T12500: Render highlighted search result body snippets in global search results views into T8646: Provide more context for search results, particularly wiki documents.
Apr 12 2017, 2:40 PM · Design, Search, Restricted Project
epriestley merged task T12500: Render highlighted search result body snippets in global search results views into T8646: Provide more context for search results, particularly wiki documents.
Apr 12 2017, 2:40 PM · Elasticsearch, Search, Feature Request
epriestley added a comment to T12500: Render highlighted search result body snippets in global search results views.

I'm just going to merge this into T8646 which broadly discusses richer search results.

Apr 12 2017, 2:40 PM · Elasticsearch, Search, Feature Request
epriestley moved T10315: Make it more clear to users how to rebuild search indexes after adding search support to custom fields from v2 to Future on the Search board.
Apr 12 2017, 2:38 PM · Custom Fields, Search, Documentation
epriestley renamed T7860: When users search for "#x y", treat "#x" as a project hashtag from Project tags can not be found via the fulltext search index to When users search for "#x y", treat "#x" as a project hashtag.
Apr 12 2017, 2:38 PM · Search
epriestley edited projects for T6906: Archived / closed objects don't have "closed" look in typeahead results, in global search typeahead only, added: Typeahead; removed Search.
Apr 12 2017, 2:36 PM · Typeahead, Contributor Onboarding, Design
epriestley moved T12493: Upgrading: Fulltext Search Services from Backlog to v2 on the Search board.
Apr 12 2017, 2:35 PM · Elasticsearch, Search, Guides
epriestley moved T12500: Render highlighted search result body snippets in global search results views from Backlog to v2 on the Search board.
Apr 12 2017, 2:35 PM · Elasticsearch, Search, Feature Request
chad added a comment to T12544: Cannot add invitees to all future instances of a recurring event.

@ivo Just a reminder this is a real install, please do not create test events on it. You can use a free instance on http://phacility.com/ if you want a clean, up to date install to test against.

Apr 12 2017, 2:28 PM · Calendar, Bug Report
epriestley closed T12244: Provide a crude way to selectively disable reads and writes of repository devices by setting flags on Almanac bindings as Wontfix.

DiffusionSSHWorkflow->execute() currently makes a proxying decision before it determines if a request is a read or write request. This needs to be reworked before we can support separate flags, since the host we'd choose to proxy the request will vary if only some hosts are readable/writable.

Apr 12 2017, 2:03 PM · Restricted Project, Almanac
epriestley added a comment to T12244: Provide a crude way to selectively disable reads and writes of repository devices by setting flags on Almanac bindings.

(For reference, see discussion elsewhere.)

Apr 12 2017, 1:34 PM · Restricted Project, Almanac
ivo created T12544: Cannot add invitees to all future instances of a recurring event.
Apr 12 2017, 12:52 PM · Calendar, Bug Report
epriestley added a comment to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.

My intended fix here is:

Apr 12 2017, 12:03 PM · Customer Impact, Typeahead, Bug Report
epriestley moved T12543: `phd` can't handle PID properly when there are multiple phabricator daemons. from Backlog to Future on the Daemons board.
Apr 12 2017, 11:53 AM · Daemons, Bug Report
epriestley triaged T12543: `phd` can't handle PID properly when there are multiple phabricator daemons. as Wishlist priority.

Running multiple different versions of Phabricator on a single host is not currently supported. We should probably handle this situation better than we do, and there is no technical reason we can't support this, but this use case is very rare.

Apr 12 2017, 11:53 AM · Daemons, Bug Report
QuietJoon updated the task description for T12543: `phd` can't handle PID properly when there are multiple phabricator daemons..
Apr 12 2017, 6:45 AM · Daemons, Bug Report
QuietJoon updated the task description for T12543: `phd` can't handle PID properly when there are multiple phabricator daemons..
Apr 12 2017, 6:35 AM · Daemons, Bug Report
QuietJoon added a comment to T12543: `phd` can't handle PID properly when there are multiple phabricator daemons..

I'v checked that

Apr 12 2017, 6:31 AM · Daemons, Bug Report
QuietJoon created T12543: `phd` can't handle PID properly when there are multiple phabricator daemons..
Apr 12 2017, 6:20 AM · Daemons, Bug Report
mcsharma1990 edited the content of Organizations Using Phabricator.
Apr 12 2017, 5:28 AM
mcsharma1990 edited the content of Organizations Using Phabricator.
Apr 12 2017, 5:27 AM

Apr 11 2017

amckinley closed T12523: Remove the ability to delete countdowns as Resolved by committing Restricted Diffusion Commit.
Apr 11 2017, 11:52 PM · Countdown
amckinley added a revision to T12523: Remove the ability to delete countdowns: Unknown Object (Differential Revision).
Apr 11 2017, 11:44 PM · Countdown
amckinley claimed T12523: Remove the ability to delete countdowns.
Apr 11 2017, 11:40 PM · Countdown
fcoelho added a comment to T12541: Unable to resolve method 'describeAutomaticCapability'.

Thanks for the quick fix!

Apr 11 2017, 11:34 PM · Bug Report
epriestley added a comment to T12542: Build a macro typeahead.

How about "i" for "image"?

Apr 11 2017, 11:29 PM · Macros, Remarkup
chad added a comment to T12542: Build a macro typeahead.

⇧ + ↓

Apr 11 2017, 11:08 PM · Macros, Remarkup
epriestley added a comment to T12542: Build a macro typeahead.

I am also keeping an eye on https://www.reddit.com/r/bonehurtingjuice/. It's still a small meme but if it takes off we're all going to be drinking bone hurting juice this time next month.

Apr 11 2017, 11:06 PM · Macros, Remarkup
chad added a comment to T12542: Build a macro typeahead.

just need to assign a trigger for the typeahead.....hmmmmmmmmm....

Apr 11 2017, 11:06 PM · Macros, Remarkup
epriestley added a comment to T12542: Build a macro typeahead.

Also our meme format is very outdated, the new meme format is:

Apr 11 2017, 10:58 PM · Macros, Remarkup
chad moved T12542: Build a macro typeahead from Backlog to Tomfoolery on the Remarkup board.
Apr 11 2017, 10:44 PM · Macros, Remarkup
chad created T12542: Build a macro typeahead.
Apr 11 2017, 10:44 PM · Macros, Remarkup
epriestley added a comment to T12541: Unable to resolve method 'describeAutomaticCapability'.

This should be fixed at HEAD of master, thanks for the report!

Apr 11 2017, 10:24 PM · Bug Report
epriestley closed T12541: Unable to resolve method 'describeAutomaticCapability' as Resolved by committing Restricted Diffusion Commit.
Apr 11 2017, 10:21 PM · Bug Report
ttapers added a comment to T12540: Conduit - Get tasks assigned to a column.

For those looking for how to associate tasks to columns - reference the following from T12074:

Apr 11 2017, 10:21 PM · Projects, Maniphest, Conduit, Feature Request
• tomekj2ee awarded T12167: Unable to edit menu items on mobile a Love token.
Apr 11 2017, 10:10 PM · Profile Menu Item, Bug Report
epriestley added a revision to T12541: Unable to resolve method 'describeAutomaticCapability': Unknown Object (Differential Revision).
Apr 11 2017, 10:00 PM · Bug Report
chad merged T12540: Conduit - Get tasks assigned to a column into T12074: conduit API extensions for subprojects and milestones..
Apr 11 2017, 9:43 PM · Feature Request
chad merged task T12540: Conduit - Get tasks assigned to a column into T12074: conduit API extensions for subprojects and milestones..
Apr 11 2017, 9:43 PM · Projects, Maniphest, Conduit, Feature Request
chad added a comment to T12541: Unable to resolve method 'describeAutomaticCapability'.

Confirmed at HEAD.

Apr 11 2017, 9:36 PM · Bug Report
fcoelho created T12541: Unable to resolve method 'describeAutomaticCapability'.
Apr 11 2017, 9:33 PM · Bug Report
ttapers created T12540: Conduit - Get tasks assigned to a column.
Apr 11 2017, 9:30 PM · Projects, Maniphest, Conduit, Feature Request
arend.danielek updated the question details for Q609: AWS Classic vs Application ELB.
Apr 11 2017, 9:08 PM
arend.danielek updated Q609: AWS Classic vs Application ELB from to AWS Classic vs Application ELB.
Apr 11 2017, 9:08 PM
matelich awarded T929: Allow two arbitrary revisions to be compared in Diffusion a Mountain of Wealth token.
Apr 11 2017, 9:00 PM · Diffusion (v3), Restricted Project
nfirmani added a comment to T12423: Bring MenuItem customization options to User Profiles.

Awesome, makes total sense to move this towards a customization of user profiles task. Thanks!

Apr 11 2017, 8:22 PM · Dashboards, Restricted Project, Profile
epriestley triaged T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name as Low priority.
Apr 11 2017, 7:40 PM · Customer Impact, Typeahead, Bug Report
epriestley added a comment to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.

D17656 is legitimately pretty hacky so this is now entangled with T11110, etc.

Apr 11 2017, 7:40 PM · Customer Impact, Typeahead, Bug Report
chad added a comment to T12423: Bring MenuItem customization options to User Profiles.

I'm going to just repurpose this into a general "allow installs to customize user profiles" task to broaden interest.

Apr 11 2017, 7:24 PM · Dashboards, Restricted Project, Profile
epriestley triaged T12539: Revision with an older "accept" showed up in "waiting on other reviewers" after "request review" as Low priority.

This is a legitimate bug but I think it's very long-standing and fixing it requires doing a lot of extra data loading to get the current diff PHID. Until someone else notices, I'd rather wait for diff PHIDs to be denormalized onto revisions.

Apr 11 2017, 7:23 PM · Differential
chad renamed T12423: Bring MenuItem customization options to User Profiles from Add link for Revisions back to profile pages to Bring MenuItem customization options to User Profiles.
Apr 11 2017, 7:23 PM · Dashboards, Restricted Project, Profile
chad added a comment to T12423: Bring MenuItem customization options to User Profiles.

We're OK leaving this open to understand impact to other installs or similar needs in customizing user profiles, but to date no other requests have happened.

Apr 11 2017, 7:20 PM · Dashboards, Restricted Project, Profile
chad added a comment to T12423: Bring MenuItem customization options to User Profiles.

I don't see a compelling reason to add a link to every Phabricator install for a single user.

Apr 11 2017, 7:19 PM · Dashboards, Restricted Project, Profile
epriestley renamed T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name from Search field in header doesn't show both tags and users with same name to Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.
Apr 11 2017, 7:13 PM · Customer Impact, Typeahead, Bug Report
epriestley added a revision to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name: Unknown Object (Differential Revision).
Apr 11 2017, 7:11 PM · Customer Impact, Typeahead, Bug Report
epriestley added a project to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name: Typeahead.

This is a sort-of-intended result of several different rules colliding to make a mess. Here's the primary rule:

Apr 11 2017, 7:09 PM · Customer Impact, Typeahead, Bug Report
nfirmani added a comment to T12423: Bring MenuItem customization options to User Profiles.

So, what's the word on this?

Apr 11 2017, 6:51 PM · Dashboards, Restricted Project, Profile
mbishopim3 added a comment to T12536: `PhabricatorRepositoryGitCommitMessageParserWorker` fails with `Got unexpected parameters: isPartial`.

Thanks. Deployed just now and confirmed working.

Apr 11 2017, 6:45 PM · Bug Report
epriestley created T12539: Revision with an older "accept" showed up in "waiting on other reviewers" after "request review".
Apr 11 2017, 6:44 PM · Differential
epriestley added a comment to T12536: `PhabricatorRepositoryGitCommitMessageParserWorker` fails with `Got unexpected parameters: isPartial`.

This should be fixed by D17654 and available at HEAD of master and stable. Thanks for the report!

Apr 11 2017, 6:43 PM · Bug Report
epriestley added a comment to T12535: Diffusion view to show the commits within a range..

This is technically supported, but not easy to find in the UI right now. Here's an example:

Apr 11 2017, 6:39 PM · Feature Request
epriestley added a comment to T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.

This reproduces for me locally. (??!?)

Apr 11 2017, 6:36 PM · Customer Impact, Typeahead, Bug Report
epriestley closed T12536: `PhabricatorRepositoryGitCommitMessageParserWorker` fails with `Got unexpected parameters: isPartial` as Resolved by committing Restricted Diffusion Commit.
Apr 11 2017, 6:29 PM · Bug Report
swisspol created T12538: Search field in header doesn't show both tags and (disabled, bot, or mailing list) users with same name.
Apr 11 2017, 6:23 PM · Customer Impact, Typeahead, Bug Report
epriestley added a revision to T12536: `PhabricatorRepositoryGitCommitMessageParserWorker` fails with `Got unexpected parameters: isPartial`: Unknown Object (Differential Revision).
Apr 11 2017, 5:56 PM · Bug Report
cos created T12537: Ambiguous display of commits in maniphest.
Apr 11 2017, 5:14 PM · Feature Request
mbishopim3 created T12536: `PhabricatorRepositoryGitCommitMessageParserWorker` fails with `Got unexpected parameters: isPartial`.
Apr 11 2017, 5:06 PM · Bug Report
cos created T12535: Diffusion view to show the commits within a range..
Apr 11 2017, 5:05 PM · Feature Request
epriestley added a comment to T11760: Phacility Cluster: Ad-Hoc Query Log.

A small subset of instances were queried with this script (below) to assess the impact/priority of T12137. This script examines fulltext search queries, but the only output is a ratio of potentially affected, unique queries first issued in the last 30 days, like this:

Apr 11 2017, 4:42 PM · Phacility
ollehar added a comment to T4139: Add support for desktop notifications.
Apr 11 2017, 2:17 PM · Aphlict
epriestley added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.

This is currently expected, although we could consider changing it.

Apr 11 2017, 11:29 AM · Bug Report (Needs Information), Differential
TafThorne added a comment to T12511: Add Support For User Definable Macro Or Template Markup to Phriction Similar To WikiMedia Templates.

Here are a few of the use cases for templates (as in {{math 2 + 2}} results in 4) that I can remember from the past.

Apr 11 2017, 11:01 AM · Phriction, Feature Request
joshuaspence added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.

When I turn on bucketing, I can't see it actually. Although I do have a lot of diffs, so it could have overheated.

Apr 11 2017, 7:22 AM · Bug Report (Needs Information), Differential
chad added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.

Where does it go when you turn on bucketing?

Apr 11 2017, 6:59 AM · Bug Report (Needs Information), Differential
chad added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.

👍🏻 😄

Apr 11 2017, 6:56 AM · Bug Report (Needs Information), Differential
joshuaspence added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.
phabricator5cf53f7b3b424f1b988a60768e1ee0bc2781c571
arcanista59cfca5f190c44403dfc7449c678a2aa1626bb4
phutilfb9e0642c4ea9065e68d2cd2b250c0fa71190e7b
Apr 11 2017, 6:39 AM · Bug Report (Needs Information), Differential
chad renamed T12534: Differential still appears in "responsible users" query after resigning as a reviewer from Differential still appears in "responsible users" query after resigning as a reviewier to Differential still appears in "responsible users" query after resigning as a reviewer.
Apr 11 2017, 5:59 AM · Bug Report (Needs Information), Differential
chad added a comment to T12534: Differential still appears in "responsible users" query after resigning as a reviewer.

Please put your version information on all bug reports. There has been a lot of churn here so we need to know you're at HEAD before trying to reproduce.

Apr 11 2017, 5:56 AM · Bug Report (Needs Information), Differential
joshuaspence created T12534: Differential still appears in "responsible users" query after resigning as a reviewer.
Apr 11 2017, 5:27 AM · Bug Report (Needs Information), Differential
• sosoqqq added a comment to T11609: Don't resize small pictures in Remarkup.
Apr 11 2017, 3:01 AM · Remarkup
epriestley changed the status of T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not from Invalid to Resolved by committing Restricted Diffusion Commit.
Apr 11 2017, 12:28 AM · Owners, Restricted Project, Bug Report
epriestley added a revision to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not: Unknown Object (Differential Revision).
Apr 11 2017, 12:10 AM · Owners, Restricted Project, Bug Report
epriestley renamed T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not from Reviewers are unable to accept for a subset of packages if they have all been rejected to Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.
Apr 11 2017, 12:08 AM · Owners, Restricted Project, Bug Report
lvital closed T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not as Invalid.

Thanks, closing this one!

Apr 11 2017, 12:07 AM · Owners, Restricted Project, Bug Report
epriestley added a comment to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.

You should have sweeping janitorial powers as a member of Community now.

Apr 11 2017, 12:05 AM · Owners, Restricted Project, Bug Report
epriestley added a member for Community: lvital.
Apr 11 2017, 12:04 AM
lvital added a comment to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.

Or I guess, you* can close this since I can't. Whoops.

Apr 11 2017, 12:03 AM · Owners, Restricted Project, Bug Report
lvital added a comment to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.

Scratch that, you're right @epriestley . We just got confused during the workflow and didn't realize jmeador previously accepted.

Apr 11 2017, 12:02 AM · Owners, Restricted Project, Bug Report

Apr 10 2017

lvital added a comment to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.

Whoops, I think I may have totally fudged the screenshots. Let me try to reproduce with proper screenshots.

Apr 10 2017, 11:39 PM · Owners, Restricted Project, Bug Report
epriestley added a comment to T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.

It looks like before step (2), jmeador can only accept for one reviewer (the package), since he's already accepted for himself. The current rule is:

Apr 10 2017, 11:21 PM · Owners, Restricted Project, Bug Report
joshuaspence added a comment to T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.

This seems to have fixed the issue. Thanks for the quick fix.

Apr 10 2017, 11:07 PM · Arcanist, Files, Bug Report
lvital created T12533: Reducing "Accept" action to not render a checkbox when only one package or project would be accepted is perhaps more confusing than not.
Apr 10 2017, 11:06 PM · Owners, Restricted Project, Bug Report
epriestley added a comment to T12515: Upgrading: File Integrity Hashing and SHA1.

T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0 discusses one side effect of these changes, it should be fixed in HEAD of master and stable now.

Apr 10 2017, 11:04 PM · Files, Guides, Security
epriestley awarded T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0 a Mountain of Wealth token.
Apr 10 2017, 11:03 PM · Arcanist, Files, Bug Report
epriestley closed T12503: Curtain sections may overflow boundaries in the UI as Resolved by committing Restricted Diffusion Commit.
Apr 10 2017, 11:02 PM · PHUI
epriestley closed T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0 as Resolved by committing Restricted Diffusion Commit.
Apr 10 2017, 11:01 PM · Arcanist, Files, Bug Report
epriestley added a comment to T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.

Thanks for that catch in D17651, let me know if you're still seeing issues after updating. I'll cherry-pick that to stable too.

Apr 10 2017, 11:00 PM · Arcanist, Files, Bug Report
epriestley added a comment to T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.

Yeah, that looks like it's consistent with the hypothesis above. I expect:

Apr 10 2017, 10:58 PM · Arcanist, Files, Bug Report
joshuaspence added a comment to T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.

Not sure if this is useful:

Apr 10 2017, 10:52 PM · Arcanist, Files, Bug Report
epriestley added a comment to T12531: Unable to upload file: failed to read 4583864320 bytes after offset 0.

Yeah, pretty sure what's happening is:

Apr 10 2017, 10:52 PM · Arcanist, Files, Bug Report