Page MenuHomePhabricator
Feed All Stories

Jun 8 2016

jonathanrseawright added a comment to T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users.
Somewhat related to type-ahead:

Is there a task open that will move archived projects to the end of the type-ahead list?

Jun 8 2016, 3:13 PM · Typeahead, Maniphest, Projects, Feature Request
epriestley closed T11111: moving storage location hint has disappeared as Wontfix.

This isn't a use case we're interested in supporting. Phabricator creates the repository in a well-defined location, not a random location.

Jun 8 2016, 3:04 PM · Feature Request
nikolay.metchev added a comment to T11111: moving storage location hint has disappeared.

I was adding a new repo to Phabricator. By default Phabricator creates the repo in a random location. I wanted to make the new repo follow the convention of all our other repos.

Jun 8 2016, 3:01 PM · Feature Request
maxie added a comment to T11112: Herald allows setting multiple values in fields that should only accept one value.

Not only this, but multiple Herald rules can conflict.

Jun 8 2016, 2:58 PM · Herald, Bug Report
maxie added a comment to T11112: Herald allows setting multiple values in fields that should only accept one value.

(...) I think these are fairly self-evident (...)

Jun 8 2016, 2:57 PM · Herald, Bug Report
jdforrester added a comment to T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users.

Yeah; I have a few times implemented a type-ahead system in the past where the already-present options are shown at the bottom to allow better accessibility for moving tags around in the list (e.g. for people that can't drag things), so that might be a better solution anyway.

Jun 8 2016, 2:56 PM · Typeahead, Maniphest, Projects, Feature Request
epriestley added a comment to T11111: moving storage location hint has disappeared.

That isn't a usable root problem description, you've just restated the original feature request.

Jun 8 2016, 2:53 PM · Feature Request
chad closed T10899: Print CSS does not fold/hide curtains for object pages like Phortune carts as Resolved by committing Restricted Diffusion Commit.
Jun 8 2016, 2:52 PM · Design, Bug Report
epriestley renamed T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users from Type-ahead should filter out results already present in the box to Type-ahead should sort results already present to the bottom of the list, like disabled users.
Jun 8 2016, 2:48 PM · Typeahead, Maniphest, Projects, Feature Request
nikolay.metchev added a comment to T11111: moving storage location hint has disappeared.

I was trying to do something that I have done several times before (change the storage location of a mercurial repo).
Up until today i always first attempted to do so by editing the repo in the GUI. The GUI up until today always told me "run such and such a command". This is no longer the case. I had to figure out the command myself.

Jun 8 2016, 2:48 PM · Feature Request
epriestley added a comment to T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users.

This creates a problem for absent-minded or new users:

Jun 8 2016, 2:47 PM · Typeahead, Maniphest, Projects, Feature Request
epriestley added a comment to T11111: moving storage location hint has disappeared.

When filing a feature request, we need you to describe your root problem, not just ask for a feature. See Describing Root Problems for help.

Jun 8 2016, 2:45 PM · Feature Request
epriestley closed T11113: Query error in Conpherence when searching for specific user as Resolved by committing Restricted Diffusion Commit.
Jun 8 2016, 2:39 PM · Conpherence, Bug Report
epriestley added a revision to T11113: Query error in Conpherence when searching for specific user: Unknown Object (Differential Revision).
Jun 8 2016, 2:37 PM · Conpherence, Bug Report
eadler added a comment to T11112: Herald allows setting multiple values in fields that should only accept one value.

Not only this, but multiple Herald rules can conflict.

Jun 8 2016, 2:33 PM · Herald, Bug Report
epriestley added a comment to T11113: Query error in Conpherence when searching for specific user.

I think I figured this one out: it's actually because more than 100 rooms are being filtered. The (`id` < 367) part is being added by pagination code, but won't activate if a user is in less than 100 rooms.

Jun 8 2016, 2:31 PM · Conpherence, Bug Report
epriestley triaged T11112: Herald allows setting multiple values in fields that should only accept one value as Wishlist priority.

We should fix these, but I think these are fairly self-evident and I think fixing them isn't trivial since I recall declining to fix them the last time I was in this code.

Jun 8 2016, 2:26 PM · Herald, Bug Report
jayvdb added a comment to T3025: Detect when browser and account timezone settings differ and prompt user to reconcile them.

My device is part of my life based in a real timezone. My OSS development and other intensive online community activity occurs in UTC, to simplify my mental calculations about other people's current time and aiding communication of timestamps and deltas.

Jun 8 2016, 2:25 PM · Prioritized, Restricted Project, User Preferences
jayvdb added a comment to T5375: Tokenizers fail in Firefox on Android.

Can reproduce in v 47.0

Jun 8 2016, 2:10 PM · Firefox, Android
jayvdb closed T9642: Unable to download images on Android as Resolved.

I tested on 4.4.2 and it worked.

Jun 8 2016, 1:56 PM · Android, Files
epriestley added a comment to T3025: Detect when browser and account timezone settings differ and prompt user to reconcile them.

I'm not sure I understand the use case for that. Do you set your client OS to local time, but prefer to use Phabricator in UTC? Offhand, this seems unusual. Do you have any particular reason for doing this (e.g., you live right next to a timezone boundary and change zones multiple times per day)?

Jun 8 2016, 1:56 PM · Prioritized, Restricted Project, User Preferences
jayvdb removed a project from T10494: Hehe: Android.
Jun 8 2016, 1:51 PM
jayvdb changed the status of T4653: Hello Task from Resolved to Invalid.
Jun 8 2016, 1:50 PM
jayvdb added a comment to T3025: Detect when browser and account timezone settings differ and prompt user to reconcile them.

Could we special case UTC, with a custom message and simpler "no change" process?

Jun 8 2016, 1:46 PM · Prioritized, Restricted Project, User Preferences
epriestley added a comment to T4653: Hello Task.

I added you to Community, so you should be able to make edits yourself now

Jun 8 2016, 1:46 PM
epriestley added a member for Community: jayvdb.
Jun 8 2016, 1:45 PM
jayvdb added a comment to T4653: Hello Task.

Because they waste people's time being looked at forever.

Jun 8 2016, 1:44 PM
epriestley added a comment to T4653: Hello Task.

Yes, but why?

Jun 8 2016, 1:41 PM
jayvdb added a comment to T4653: Hello Task.

Can we move tasks like this out of real projects? To a junk project?

Jun 8 2016, 1:40 PM
epriestley added a comment to T8282: What is the purpose of creating an admin account?.

Specifically, you saw this page, and decided that you were not in the right place?

Jun 8 2016, 1:40 PM · Setup
jayvdb added a watcher for Android: jayvdb.
Jun 8 2016, 1:37 PM
asherkin updated the task description for T11113: Query error in Conpherence when searching for specific user.
Jun 8 2016, 1:22 PM · Conpherence, Bug Report
asherkin created T11113: Query error in Conpherence when searching for specific user.
Jun 8 2016, 1:21 PM · Conpherence, Bug Report
maxie created T11112: Herald allows setting multiple values in fields that should only accept one value.
Jun 8 2016, 12:47 PM · Herald, Bug Report
nikolay.metchev created T11111: moving storage location hint has disappeared.
Jun 8 2016, 11:03 AM · Feature Request
jdforrester added a comment to T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users.

(Low priority IMO.)

Jun 8 2016, 10:25 AM · Typeahead, Maniphest, Projects, Feature Request
jdforrester created T11110: Type-ahead should sort results already present to the bottom of the list, like disabled users.
Jun 8 2016, 10:25 AM · Typeahead, Maniphest, Projects, Feature Request
asherkin added a comment to T8282: What is the purpose of creating an admin account?.

Was there any indication that the form wasn't working / wouldn't have worked?

Jun 8 2016, 9:37 AM · Setup
aklapper added Q387: Cannot clone repository hosted on Phabricator over HTTP (Answer 390).
Jun 8 2016, 9:30 AM
• mayuroks added a comment to T8282: What is the purpose of creating an admin account?.

Hi,
Thanks for responding. I was evaluating phabricator on my Ubuntu machine. I was certain the UI user registration wont work because email wont be sent out. So I went to terminal and used ./accountadmin to create an admin user.

Jun 8 2016, 7:27 AM · Setup
cwlin0416 updated the task description for T11109: Strings of Owner and Herald applcaitons cannot translate..
Jun 8 2016, 7:17 AM · Herald, Owners, Localization, Bug Report
cwlin0416 created T11109: Strings of Owner and Herald applcaitons cannot translate..
Jun 8 2016, 7:14 AM · Herald, Owners, Localization, Bug Report
hach-que added a comment to T11105: ExecFuture performs blocking reads on Windows, causing deadlocks with `arc lint`.

This is solved with D11688, but it needs to be turned on globally (at the moment it's a per call option, which was added so that internally our custom unit test runner wouldn't deadlock).

Jun 8 2016, 5:14 AM · Windows, Arcanist, Lint, Bug Report
chad added a revision to T10899: Print CSS does not fold/hide curtains for object pages like Phortune carts: Unknown Object (Differential Revision).
Jun 8 2016, 3:51 AM · Design, Bug Report
chad closed T11046: Create Countdown button on new installs links to wrong place as Resolved by committing Restricted Diffusion Commit.
Jun 8 2016, 3:12 AM · Countdown, Bug Report
chad added a revision to T11046: Create Countdown button on new installs links to wrong place: Unknown Object (Differential Revision).
Jun 8 2016, 1:20 AM · Countdown, Bug Report
epriestley merged task T11108: "Create a countdown" button has incorrect href into T11046: Create Countdown button on new installs links to wrong place.
Jun 8 2016, 12:01 AM · Countdown, Bug Report
epriestley merged T11108: "Create a countdown" button has incorrect href into T11046: Create Countdown button on new installs links to wrong place.
Jun 8 2016, 12:01 AM · Countdown, Bug Report

Jun 7 2016

SamStudio8 created T11108: "Create a countdown" button has incorrect href.
Jun 7 2016, 11:57 PM · Countdown, Bug Report
epriestley added a comment to T11107: If you forget your password you can never reset it.

This should be resolved at HEAD of master. To get it to work you may need to update and then go through the password reset process again.

Jun 7 2016, 10:02 PM · Bug Report
epriestley added a revision to T6916: Allow video embed in remarkup: Unknown Object (Differential Revision).
Jun 7 2016, 9:55 PM · Prioritized, Remarkup
epriestley added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

I'm sure there are still some cases where we don't do a particularly good job, but let me know when you run into them.

Jun 7 2016, 9:51 PM · Prioritized, Transactions
MZMcBride added a comment to T7643: Improve prose diffs (was: description changes don't generate usable diffs).

Diffs should be less choppy now, here's the improved version of the above change:

Screen Shot 2016-06-07 at 9.01.32 AM.png (508×621 px, 67 KB)

Jun 7 2016, 9:45 PM · Prioritized, Transactions
quiddity added a comment to T6916: Allow video embed in remarkup.

I have no idea if this is MP4 or not:

Jun 7 2016, 9:43 PM · Prioritized, Remarkup
epriestley moved T6916: Allow video embed in remarkup from The Queue to Paused on the Prioritized board.

I think this one works:

Jun 7 2016, 9:24 PM · Prioritized, Remarkup
avivey changed the visibility for F1679211: aDmqyOZ_460sv.mp4.
Jun 7 2016, 9:24 PM
avivey changed the visibility for F1679211: aDmqyOZ_460sv.mp4.
Jun 7 2016, 9:23 PM
epriestley added a comment to T6916: Allow video embed in remarkup.

I have no idea if this is MP4 or not:

Jun 7 2016, 9:16 PM · Prioritized, Remarkup
epriestley closed T11107: If you forget your password you can never reset it as Resolved by committing Restricted Diffusion Commit.
Jun 7 2016, 9:07 PM · Bug Report
epriestley added a revision to T11107: If you forget your password you can never reset it: Unknown Object (Differential Revision).
Jun 7 2016, 9:06 PM · Bug Report
epriestley claimed T11107: If you forget your password you can never reset it.
Jun 7 2016, 9:06 PM · Bug Report
avivey changed the visibility for F1679211: aDmqyOZ_460sv.mp4.
Jun 7 2016, 9:02 PM
chad updated subscribers of T11107: If you forget your password you can never reset it.

I think maybe there is a bug here also though with EditEngine. @epriestley ?

Jun 7 2016, 9:02 PM · Bug Report
chad added a comment to T11107: If you forget your password you can never reset it.

./bin/auth

Jun 7 2016, 8:57 PM · Bug Report
rbalik created T11107: If you forget your password you can never reset it.
Jun 7 2016, 8:53 PM · Bug Report
epriestley added a revision to T7643: Improve prose diffs (was: description changes don't generate usable diffs): Unknown Object (Differential Revision).
Jun 7 2016, 8:45 PM · Prioritized, Transactions
epriestley added a revision to T7643: Improve prose diffs (was: description changes don't generate usable diffs): Unknown Object (Differential Revision).
Jun 7 2016, 8:43 PM · Prioritized, Transactions
chad moved T10899: Print CSS does not fold/hide curtains for object pages like Phortune carts from Backlog to Active on the Design board.
Jun 7 2016, 8:29 PM · Design, Bug Report
chad moved T11077: Redesign Comment / Action Box from Backlog to Active on the Design board.
Jun 7 2016, 8:29 PM · Design
chad moved T11096: Redesign main header / quick create / search from Backlog to Active on the Design board.
Jun 7 2016, 8:29 PM · Design
chad moved T10055: Redesign Hovercards from Active to Backlog on the Design board.
Jun 7 2016, 8:29 PM · Restricted Project, Hovercards, Design
chad moved T4011: Improve Projects list view / search result layout from Active to Backlog on the Design board.
Jun 7 2016, 8:28 PM · Design, Projects (v3)
chad moved T10286: Design a Milestone Tag from Active to Backlog on the Design board.
Jun 7 2016, 8:28 PM · Restricted Project, Design, Projects (v3)
chad moved T10208: Document something about Design Feedback from Active to Backlog on the Design board.
Jun 7 2016, 8:28 PM · Documentation, Design
lawrencet added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

Yes - that definitely makes sense. Thanks!

Jun 7 2016, 7:17 PM
chad added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

At least with Flags you can still review it after landing and raise any concerns.

Jun 7 2016, 7:15 PM
epriestley added a revision to T7643: Improve prose diffs (was: description changes don't generate usable diffs): Unknown Object (Differential Revision).
Jun 7 2016, 7:15 PM · Prioritized, Transactions
chad added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

Yeah unfortunately we don't have not rules for queries, just not much interest. Herald can be used to auto-flag like you suggest. It's not a perfect solution for what you want, but I think gets most of it in an absolute way. My main concern about just using a query is some sort of race condition where your sent a review, it's reviewed and landed while you're out in a meeting.

Jun 7 2016, 7:14 PM
asherkin added a comment to T8282: What is the purpose of creating an admin account?.

bin/auth recover is a general-purpose flow not specific to this case.

Jun 7 2016, 6:42 PM · Setup
chad added a comment to T8282: What is the purpose of creating an admin account?.

I've never needed to recover an account. Can you give us details about how you ended up in a place where you felt that was necessary?

Jun 7 2016, 6:41 PM · Setup
avivey moved T9069: arc patch can't handle binary file correctly from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 7 2016, 6:34 PM · Differential, Restricted Project, Arcanist
• mayuroks added a comment to T8282: What is the purpose of creating an admin account?.

I faced the same issue too. As a user, after setting up a web UI why go back to terminal and type

./bin/auth recover <username>

and then come back to web UI to login? Once the setup is completed, a user would feel that the job is done and now he can login in peace.

Jun 7 2016, 6:31 PM · Setup
eadler moved T11090: Marking comments as done in Differential isn't always saved correctly from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 7 2016, 6:20 PM · Restricted Project, Differential, Bug Report
eadler moved T1894: Support commenting on Phriction documents from Restricted Project Column to Restricted Project Column on the Restricted Project board.
Jun 7 2016, 6:19 PM · Restricted Project, Restricted Project, Phriction
jonathanrseawright awarded T10612: Editing a panel causes it to duplicate in dashboard a Like token.
Jun 7 2016, 6:19 PM · Dashboards (v2), Bug Report
eadler added a project to T11090: Marking comments as done in Differential isn't always saved correctly: Restricted Project.
Jun 7 2016, 6:19 PM · Restricted Project, Differential, Bug Report
jonathanrseawright added a comment to T10612: Editing a panel causes it to duplicate in dashboard.

I saw this today, we just created a new panel

Jun 7 2016, 6:19 PM · Dashboards (v2), Bug Report
epriestley merged T6674: Support video MIME types and distinguish between audio and video OGG files into T6916: Allow video embed in remarkup.
Jun 7 2016, 6:06 PM · Prioritized, Remarkup
epriestley merged task T6674: Support video MIME types and distinguish between audio and video OGG files into T6916: Allow video embed in remarkup.
Jun 7 2016, 6:06 PM · Files
epriestley added a comment to T6674: Support video MIME types and distinguish between audio and video OGG files.

I'm going to merge this into T6916. That implements a reasonable behavior for application/ogg (assume video by default, allow users to embed as audio instead with media="audio") and browser seem to do fine when audio is embedded with <video /> anyway, so I suspect we'll be in reasonable shape after that and not need to try to determine the contents of OGG containers in a more sophisticated way.

Jun 7 2016, 6:06 PM · Files
epriestley moved T6916: Allow video embed in remarkup from Backlog to The Queue on the Prioritized board.
Jun 7 2016, 6:04 PM · Prioritized, Remarkup
epriestley claimed T6916: Allow video embed in remarkup.
Jun 7 2016, 6:04 PM · Prioritized, Remarkup
epriestley closed T11106: Prueba as Invalid.
Jun 7 2016, 5:40 PM · Abuse
avivey added a project to T9069: arc patch can't handle binary file correctly: Restricted Project.

I've re-created 2 sub-cases of "arc patch with binary files":

  1. deleting files with --skip-binaries D15972: When removing large binary files, specifying --skip-binaries, the patch fails
  2. rename + move D16070: When git detects a file as "renamed and moved", the patch fails. If the modification is huge, it's considered "delete and add", which does work.
Jun 7 2016, 5:24 PM · Differential, Restricted Project, Arcanist
lawrencet closed Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me? as resolved.
Jun 7 2016, 5:23 PM
lawrencet added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

Thanks for the suggestion! I tried the search you suggested, but I think that pulls up both revisions that I've accepted and revisions that I haven't yet accepted.

Jun 7 2016, 5:17 PM
chad added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

Overall though this feels arbitrary if you need to be performing reviews. That is, it's not really consistent or programatic that you'll catch something. Prefer using owners/herald/blocking reviewers to enforce better review policy.

Jun 7 2016, 5:14 PM
chad added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

I would "Flag for Later" the review then if this is an individual desire or create a new search with something like "Reviewer = me and status = accepted"

Jun 7 2016, 5:12 PM
lawrencet added a comment to Q415: Can I list Differential revisions I'm a reviewer for that has been accepted by someone else but not by me?.

Not exactly - we tend to list multiple reviewers on changes, but we don't formally require multiple reviewers to accept before landing changes.

Jun 7 2016, 5:08 PM
• cristomanuel created T11106: Prueba.
Jun 7 2016, 5:06 PM · Abuse