Page MenuHomePhabricator
Feed All Stories

Apr 18 2017

amckinley claimed T11476: Record transaction histories when users edit application capabilities.
Apr 18 2017, 9:30 PM · Contributor Onboarding, Restricted Project, Applications, Feature Request
chad added a comment to T12572: Spend a week making Conpherence less bad.

very noisy

Apr 18 2017, 8:53 PM · Conpherence (v4)
chad closed T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters as Invalid.

Please read and follow Contributing Bug Reports and Providing Reproduction Steps.

Apr 18 2017, 8:48 PM · Bug Report
epriestley set the audio for Image Macro "wellmet" to play once.
Apr 18 2017, 8:42 PM
epriestley attached audio to Image Macro "wellmet": F4914286: wellmet.mp3.
Apr 18 2017, 8:40 PM
epriestley changed the image for Image Macro "wellmet".
Apr 18 2017, 8:40 PM
epriestley added a revision to T7567: Support sound in Conpherence: Unknown Object (Differential Revision).
Apr 18 2017, 8:27 PM · Conpherence (v4)
amckinley closed T12587: Migrate file deletion to modular transactions as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 8:01 PM · Files
cos created T12588: create subtask side menu to allow creation of subtasks of different types to the parent task.
Apr 18 2017, 7:59 PM · Feature Request
amckinley added a revision to T12587: Migrate file deletion to modular transactions: Unknown Object (Differential Revision).
Apr 18 2017, 7:23 PM · Files
epriestley added a comment to T12587: Migrate file deletion to modular transactions.

We'd also have to add locking to prevent a race like this:

Apr 18 2017, 7:11 PM · Files
epriestley added a comment to T12587: Migrate file deletion to modular transactions.

My gut is to wait for users to hit those cases before building solutions for them since I think it's unlikely any of that will ever turn into a real problem.

Apr 18 2017, 7:07 PM · Files
amckinley added a comment to T12587: Migrate file deletion to modular transactions.

@epriestley should it be possible to un-delete a File, the same way it's possible to archive and then un-archive a Paste? The only benefit I can see is that in an emergency where someone accidentally kicked off a delete for a bunch of files they didn't intend to, someone could stop the daemons and perform an undelete operation before restarting the daemons (instead of flushing the worker queue). I actually mostly coded it this way already by following along with PhabricatorPasteStatusTransaction.

Apr 18 2017, 6:47 PM · Files
epriestley added a revision to T12566: After an Aphlict reconnect, dirty realtime UI state: Unknown Object (Differential Revision).
Apr 18 2017, 6:31 PM · Aphlict, Conpherence (v4)
epriestley added a revision to T12563: When a client reconnects to Aphlict, replay recent messages: Unknown Object (Differential Revision).
Apr 18 2017, 6:31 PM · Aphlict
amckinley updated the task description for T12587: Migrate file deletion to modular transactions.
Apr 18 2017, 6:10 PM · Files
amckinley created T12587: Migrate file deletion to modular transactions.
Apr 18 2017, 6:10 PM · Files
amckinley closed T10828: File deletion should be queued up and run by daemons as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 6:09 PM · Contributor Onboarding, Daemons, Files, Bug Report
amckinley created P2040 (An Untitled Masterwork).
Apr 18 2017, 5:20 PM
johnny-bit created T12586: Support for multiple "Move on Workboard" actions in single Maniphest update.
Apr 18 2017, 5:11 PM · Maniphest, Workboards, Feature Request
epriestley closed T12576: Quoting timeline objects without calling setQuoteRef() results in strings like "In null, <user> wrote" as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 4:51 PM · Maniphest
epriestley closed T12579: ./bin/mail show-outbound command should type check <id> as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 4:51 PM · Mail, Contributor Onboarding
epriestley added a revision to T7567: Support sound in Conpherence: Unknown Object (Differential Revision).
Apr 18 2017, 4:38 PM · Conpherence (v4)
epriestley added a project to T12579: ./bin/mail show-outbound command should type check <id>: Mail.
Apr 18 2017, 4:08 PM · Mail, Contributor Onboarding
epriestley added a revision to T12576: Quoting timeline objects without calling setQuoteRef() results in strings like "In null, <user> wrote": Unknown Object (Differential Revision).
Apr 18 2017, 4:07 PM · Maniphest
epriestley claimed T12576: Quoting timeline objects without calling setQuoteRef() results in strings like "In null, <user> wrote".
Apr 18 2017, 4:05 PM · Maniphest
epriestley added a revision to T12579: ./bin/mail show-outbound command should type check <id>: Unknown Object (Differential Revision).
Apr 18 2017, 4:02 PM · Mail, Contributor Onboarding
epriestley claimed T12579: ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 3:59 PM · Mail, Contributor Onboarding
amckinley closed T8788: Allow querying for files by name as Resolved by committing Restricted Diffusion Commit.
Apr 18 2017, 3:38 PM · Contributor Onboarding, Files
ezyang updated Q610: What is arc diff --verbatim supposed to do, actually? from to What is arc diff --verbatim supposed to do, actually?.
Apr 18 2017, 3:27 PM
amckinley added a revision to T8788: Allow querying for files by name: Unknown Object (Differential Revision).
Apr 18 2017, 3:24 PM · Contributor Onboarding, Files
chad added a comment to T12584: How do I edit the subtype of a task?.

Conpherence and Ponder are the best avenues to ask questions about Phabricator, thanks!

Apr 18 2017, 3:20 PM · Feature Request
chad created E1476: School Field Trip.
Apr 18 2017, 3:10 PM
epriestley merged task T12580: Provide Conduit API to retrieve and update comments from differentials into T12495: Provide a conduit method to retrieve inline comments associated with a diff.
Apr 18 2017, 1:45 PM · Feature Request
epriestley merged T12580: Provide Conduit API to retrieve and update comments from differentials into T12495: Provide a conduit method to retrieve inline comments associated with a diff.
Apr 18 2017, 1:45 PM · Feature Request
epriestley closed T12585: Unable to create differential as Invalid.

Bug reports require reproduction steps. See Contributing Bug Reports and Providing Reproduction Steps.

Apr 18 2017, 1:44 PM · Bug Report
epriestley closed T12584: How do I edit the subtype of a task? as Resolved.

See T12314#214002.

Apr 18 2017, 1:43 PM · Feature Request
atul.agrawal9911 added a comment to T12585: Unable to create differential .

This is happening because my project directory not contains .jshintrc file

Apr 18 2017, 12:46 PM · Bug Report
atul.agrawal9911 added a comment to T12585: Unable to create differential .

my .arclint file

Apr 18 2017, 12:23 PM · Bug Report
atul.agrawal9911 created T12585: Unable to create differential .
Apr 18 2017, 12:22 PM · Bug Report
atul.agrawal9911 added a comment to T12579: ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 12:13 PM · Mail, Contributor Onboarding
Nick.Sephton created T12584: How do I edit the subtype of a task?.
Apr 18 2017, 9:20 AM · Feature Request
avivey closed T12582: test as Invalid.
Apr 18 2017, 7:45 AM · Android
• 94else merged T12583: test into T12582: test.
Apr 18 2017, 7:21 AM · Android
• 94else merged task T12583: test into T12582: test.
Apr 18 2017, 7:21 AM · Bug Report
• 94else created T12583: test.
Apr 18 2017, 7:19 AM · Bug Report
• 94else created T12582: test.
Apr 18 2017, 7:18 AM · Android
• 94else added a comment to T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.

home_aboutus_nor@2x.png (104×104 px, 491 KB)

Apr 18 2017, 7:15 AM · Bug Report
tavvva added a comment to T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.

The attached patch seems to fix the issue ... hopefully with no drawbacks. We gonna test that for few weeks.

Apr 18 2017, 7:02 AM · Bug Report
tavvva created T12581: The "Recent Activity" feed flooded with duplicates because of invalid UTF-8 characters.
Apr 18 2017, 6:47 AM · Bug Report
bcooksley added a project to T11073: Allow the user to specify a date in a different timezone using date controls: KDE.

KDE has some interest in this as our contributors are distributed around the world.

Apr 18 2017, 6:04 AM · KDE, Calendar, Feature Request
uraveragechump created T12580: Provide Conduit API to retrieve and update comments from differentials.
Apr 18 2017, 6:01 AM · Feature Request
chad placed T12579: ./bin/mail show-outbound command should type check <id> up for grabs.
Apr 18 2017, 5:51 AM · Mail, Contributor Onboarding
chad updated the task description for T12579: ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 5:51 AM · Mail, Contributor Onboarding
chad renamed T12579: ./bin/mail show-outbound command should type check <id> from Error while using ./bin/mail show-outbound command to ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 5:50 AM · Mail, Contributor Onboarding
chad reopened T12579: ./bin/mail show-outbound command should type check <id> as "Open".

Actually I'll leave this open if someone wants to contribute a patch.

Apr 18 2017, 5:49 AM · Mail, Contributor Onboarding
chad closed T12579: ./bin/mail show-outbound command should type check <id> as Invalid.
Apr 18 2017, 5:46 AM · Mail, Contributor Onboarding
atul.agrawal9911 added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

It should give me warning if i am using string value for my id part.

Apr 18 2017, 5:43 AM · Mail, Contributor Onboarding
chad added a comment to T12579: ./bin/mail show-outbound command should type check <id>.
chad@phac-dev:/var/www/html/dev/phabricator$ ./bin/mail show-outbound --id 1
PROPERTIES
ID: 1
Status: sent
Related PHID: PHID-USER-hh5jvrt4dfkdzciswub6
Message:
Apr 18 2017, 5:42 AM · Mail, Contributor Onboarding
atul.agrawal9911 added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

i am trying this command
./bin/mail show-outbound --id <my-email-id>

Apr 18 2017, 5:42 AM · Mail, Contributor Onboarding
chad added a comment to T12579: ./bin/mail show-outbound command should type check <id>.

Please provide complete steps to reproduce the error. Based on the information above I cannot reproduce anything.

Apr 18 2017, 5:40 AM · Mail, Contributor Onboarding
atul.agrawal9911 created T12579: ./bin/mail show-outbound command should type check <id>.
Apr 18 2017, 5:38 AM · Mail, Contributor Onboarding
cspeckmim awarded T12572: Spend a week making Conpherence less bad a Evil Spooky Haunted Tree token.
Apr 18 2017, 2:42 AM · Conpherence (v4)
epriestley added a comment to T12566: After an Aphlict reconnect, dirty realtime UI state.

Conpherence is repainting after D17713 but notification counts aren't, yet.

Apr 18 2017, 1:04 AM · Aphlict, Conpherence (v4)
epriestley closed T12568: Write an Aphlict client debugging console as Resolved.

This could be prettier, but it's functional and has buttons you can click.

Apr 18 2017, 1:03 AM · Aphlict
epriestley closed T12568: Write an Aphlict client debugging console, a subtask of T12573: Improve reliability of Conpherence connections, as Resolved.
Apr 18 2017, 1:03 AM · Conpherence (v4)
epriestley added a comment to T8788: Allow querying for files by name.

For the migration, let me just publish "run this command if you want to search existing files" in the change log for now, and then we can do the real migration if we see confusion about it. I bet 99% of the time you want to search for something you recently uploaded so the index will be functionally useful in a week even if you miss the guidance in the changelog.

Apr 18 2017, 12:47 AM · Contributor Onboarding, Files
epriestley added a comment to T8788: Allow querying for files by name.

To make the indexing work for new stuff, I think you can do this:

Apr 18 2017, 12:42 AM · Contributor Onboarding, Files
amckinley added a comment to T8788: Allow querying for files by name.

@epriestley I just landed D17702, without really tackling the problem of files being created in non-standard code paths. I searched the codebase for PhabricatorFile::initializeNewFile() and didn't find anything scary-looking; do you have any specific examples that might need updating?

Apr 18 2017, 12:41 AM · Contributor Onboarding, Files
amckinley added a revision to T8788: Allow querying for files by name: Unknown Object (Differential Revision).
Apr 18 2017, 12:37 AM · Contributor Onboarding, Files
gregprice added a comment to T10640: Allow application queries to be promoted as global search modes.

Just in my personal experience, I think it was sometime in the last 6 months that I realized Phabricator has a concept of "application search" and "global search", and that a lot of the searches I'd been doing were the latter and that's why they were so limited in functionality. I'd been using Phabricator for years and always had the sense that search didn't work very well, and that there were different forms I might get for it with different sets of fields, and it was unpredictable which one I'd get; but it wasn't until I became responsible for actually running Phabricator and started paying much closer attention to its concepts that I twigged to that pattern as the explanation for which searches got which form.

Apr 18 2017, 12:33 AM · Search, Feature Request
amckinley created P2039 (An Untitled Masterwork).
Apr 18 2017, 12:30 AM
epriestley added a revision to T12573: Improve reliability of Conpherence connections: Unknown Object (Differential Revision).
Apr 18 2017, 12:02 AM · Conpherence (v4)

Apr 17 2017

joshuaspence updated the task description for T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.
Apr 17 2017, 11:15 PM · Differential, Herald, Bug Report
chad added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

ehhh, not sure about this. It's not worse, like I can pinch to zoom in Safari, but not Chrome.

Apr 17 2017, 11:15 PM · Mobile
chad added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

Rolling, so we'll see how accurate the simulator was.

Apr 17 2017, 11:00 PM · Mobile
epriestley added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

I am just intrigued that user-scalable=no actually means "somewhat user-scalable".

Apr 17 2017, 10:59 PM · Mobile
epriestley added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

No, I hate all zooming.

Apr 17 2017, 10:58 PM · Mobile
epriestley added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

(I am glad to see the auto-zoom-on-controls gone and don't care about any other zoom feature.)

Apr 17 2017, 10:58 PM · Mobile
chad added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

Right. Do you use that?

Apr 17 2017, 10:58 PM · Mobile
epriestley added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

Oh -- but double-tap-to-zoom doesn't? 😕

Apr 17 2017, 10:58 PM · Mobile
chad added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

I tested pinch to zoom, which still works. We just turn off the auto-zoom. I'll roll the server here and give a more thorough shakedown.

Apr 17 2017, 10:57 PM · Mobile
epriestley added a comment to T12100: Consider removing "maximum-scale" from "viewport" tag.

(Specifically, D17714 added user-scalable=no, which probably does the opposite of what this requested.)

Apr 17 2017, 10:56 PM · Mobile
joshuaspence added a comment to T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.

Hmm ok. I think that I'm not usually listed explicitly as a reviewer, which means that Reviewers: #some_project_that_i_am_a_member_of doesn't include me. Would you be in favor of adding a new [Differential was reviewed by] field to Herald?

Apr 17 2017, 10:56 PM · Differential, Herald, Bug Report
david.sumolight added a comment to T11073: Allow the user to specify a date in a different timezone using date controls.

I originally posted this feature suggestion because I needed to put a tradeshow event in the calendar. The tradeshow took place in another time zone, and when I got there I realized that the calendar entry was on the wrong day. I also frequently schedule conference calls with clients and suppliers in other time zones.

Apr 17 2017, 10:56 PM · KDE, Calendar, Feature Request
epriestley closed T12564: Handle duplicate realtime messages from Aphlict gracefully, a subtask of T12573: Improve reliability of Conpherence connections, as Resolved.
Apr 17 2017, 10:55 PM · Conpherence (v4)
epriestley closed T12564: Handle duplicate realtime messages from Aphlict gracefully as Resolved by committing Restricted Diffusion Commit.
Apr 17 2017, 10:55 PM · Aphlict
chad closed T12100: Consider removing "maximum-scale" from "viewport" tag as Resolved.

Removed.

Apr 17 2017, 10:55 PM · Mobile
epriestley closed T12563: When a client reconnects to Aphlict, replay recent messages as Resolved by committing Restricted Diffusion Commit.
Apr 17 2017, 10:54 PM · Aphlict
epriestley closed T12563: When a client reconnects to Aphlict, replay recent messages, a subtask of T12573: Improve reliability of Conpherence connections, as Resolved.
Apr 17 2017, 10:54 PM · Conpherence (v4)
epriestley closed T12567: After an Aphlict disconnect, retry the first reconnect without a delay as Resolved by committing Restricted Diffusion Commit.
Apr 17 2017, 10:53 PM · Aphlict
epriestley closed T12567: After an Aphlict disconnect, retry the first reconnect without a delay, a subtask of T12573: Improve reliability of Conpherence connections, as Resolved.
Apr 17 2017, 10:53 PM · Conpherence (v4)
epriestley added a comment to T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.

I don't think this behavior has changed. This has always meant "reviewers", not "reviewers who accepted the revision".

Apr 17 2017, 10:50 PM · Differential, Herald, Bug Report
joshuaspence created T12578: Herald rule for "Differential reviewers do not include" didn't fire when I expected it to.
Apr 17 2017, 10:48 PM · Differential, Herald, Bug Report
epriestley added a revision to T12566: After an Aphlict reconnect, dirty realtime UI state: Unknown Object (Differential Revision).
Apr 17 2017, 10:14 PM · Aphlict, Conpherence (v4)
epriestley added a revision to T12564: Handle duplicate realtime messages from Aphlict gracefully: Unknown Object (Differential Revision).
Apr 17 2017, 9:46 PM · Aphlict
chad closed T12577: Icon buttons in Calendar for "Next Month" / "Previous Month" are aligned awkwardly as Resolved by committing Restricted Diffusion Commit.
Apr 17 2017, 9:37 PM · PHUI, Design, Calendar
chad added a revision to T12577: Icon buttons in Calendar for "Next Month" / "Previous Month" are aligned awkwardly: Unknown Object (Differential Revision).
Apr 17 2017, 9:35 PM · PHUI, Design, Calendar
epriestley added a revision to T12563: When a client reconnects to Aphlict, replay recent messages: Unknown Object (Differential Revision).
Apr 17 2017, 9:28 PM · Aphlict
chad claimed T12577: Icon buttons in Calendar for "Next Month" / "Previous Month" are aligned awkwardly.
Apr 17 2017, 9:00 PM · PHUI, Design, Calendar