Page MenuHomePhabricator
Feed Advanced Search

Jun 1 2021

bbrdaric awarded Changelog a Heartbreak token.
Jun 1 2021, 8:41 AM

Aug 13 2019

jcox awarded Changelog a Love token.
Aug 13 2019, 4:30 PM

Apr 19 2019

remusvrm awarded Changelog a Like token.
Apr 19 2019, 7:09 PM

Jan 6 2018

cspeckmim awarded Changelog a Mountain of Wealth token.
Jan 6 2018, 4:47 PM

Feb 15 2017

nornagon added a comment to T11432: Ability to exclude self ("Current Viewer") from Feed panels.

Yep, this is no longer a problem with the new audit bucketing. 馃弲

Feb 15 2017, 7:11 PMFeed, Feature Request

Jan 26 2017

nornagon added a comment to T11432: Ability to exclude self ("Current Viewer") from Feed panels.

This would also be useful for the Audit dashboard, e.g. to build the query "what audits am I responsible for that I didn't author?" (given that the viewer belongs to a project which is responsible for auditing the commit).

Jan 26 2017, 10:59 PMFeed, Feature Request

Dec 6 2016

nornagon added a comment to T11953: Herald rule for commits on autoclose branch didn't trigger when a commit was first pushed to a branch, then to master (fast-forward).

Purely organizational. I don't believe there's anything technically that Phabricator can't do, but we're just trying it out and don't want to move everything over before we're ready to commit.

Dec 6 2016, 12:47 AMDiffusion, Herald, Bug Report

Dec 5 2016

nornagon added a comment to T11952: Daemons don't notice changes to global default settings.

Sounds like this can safely be closed as PEBKAC

Dec 5 2016, 11:59 PMBug Report
nornagon added a comment to T11953: Herald rule for commits on autoclose branch didn't trigger when a commit was first pushed to a branch, then to master (fast-forward).

FWIW, in this particular case our repository is hosted on GitHub, so lovely automagic in Phabricator wouldn't solve my specific case.

Dec 5 2016, 11:58 PMDiffusion, Herald, Bug Report
nornagon added a comment to T11953: Herald rule for commits on autoclose branch didn't trigger when a commit was first pushed to a branch, then to master (fast-forward).

I think it is the same as T6522?

Dec 5 2016, 11:24 PMDiffusion, Herald, Bug Report
nornagon created T11953: Herald rule for commits on autoclose branch didn't trigger when a commit was first pushed to a branch, then to master (fast-forward).
Dec 5 2016, 11:19 PMDiffusion, Herald, Bug Report
nornagon added a comment to T11952: Daemons don't notice changes to global default settings.

Your logic there w/ dumping the prefs is definitely convincing. Perhaps I hallucinated this and it was actually that I did something in a different order, e.g. I disabled "A commit is created", then restarted the daemons, then disabled "A commit has auditors added" and misattributed causality (where the real issue was that disabling "A commit is created" is not sufficient to stop receiving emails when a commit is created that you're an auditor on).

Dec 5 2016, 10:32 PMBug Report
nornagon updated the task description for T11952: Daemons don't notice changes to global default settings.
Dec 5 2016, 10:07 PMBug Report
nornagon created T11952: Daemons don't notice changes to global default settings.
Dec 5 2016, 10:04 PMBug Report

Dec 2 2016

nornagon awarded T10978: Modernize Audit a Like token.
Dec 2 2016, 1:51 AMAudit

Aug 23 2016

nornagon created T11521: Request Review action should clear "accepted older" status.
Aug 23 2016, 6:41 PMRestricted Project, Differential, Bug Report

Mar 22 2016

nornagon awarded T10363: Reorient Doorkeeper publishing around transactions instead of feed stories a Piece of Eight token.
Mar 22 2016, 10:59 PMNuance, Restricted Project, Feed, Doorkeeper
nornagon added a comment to T5462: How do I publish Phabricator events into remote systems?.

Awesome! Thanks for the quick reply @epriestley :)

Mar 22 2016, 10:57 PMGuides, Doorkeeper, Feed, Herald
nornagon added a comment to T5462: How do I publish Phabricator events into remote systems?.

The data available from feed.http-hooks or feed.query is somewhat limited:

Mar 22 2016, 10:54 PMGuides, Doorkeeper, Feed, Herald

Nov 5 2015

nornagon added a comment to T8348: `arc download` doesn't support chunked downloads.

Could the server do the chunk stitching and stream the response to the
client, rather than fetching it all up front?

Nov 5 2015, 3:16 PMRestricted Project, Files, Arcanist

Nov 4 2015

nornagon added a comment to T8348: `arc download` doesn't support chunked downloads.

This might be because the file in question is ~150MB, so it's probably trying to download the whole thing and base64 it (?!) before spitting it back.

Nov 4 2015, 7:58 PMRestricted Project, Files, Arcanist
nornagon added a comment to T8348: `arc download` doesn't support chunked downloads.

I'm also seeing this issue, even just using curl -d api.token=... https://phabricator.url/api/file.download

Nov 4 2015, 7:53 PMRestricted Project, Files, Arcanist

Oct 25 2015

CaptSpot awarded Changelog a Like token.
Oct 25 2015, 9:38 AM

Oct 23 2015

vinzent awarded Changelog a Like token.
Oct 23 2015, 9:26 AM

Aug 19 2015

nornagon awarded T8659: Allow linking to an external build system from Harbormaster a Piece of Eight token.
Aug 19 2015, 9:20 PMPrioritized, Harbormaster
nornagon added a comment to T8659: Allow linking to an external build system from Harbormaster.

Awesome! Thanks @epriestley :)

Aug 19 2015, 9:20 PMPrioritized, Harbormaster

Jul 24 2015

nornagon created T8948: /settings/panel/emailpreferences throws "FATAL ERROR".
Jul 24 2015, 4:10 PMUser Preferences
nornagon added a comment to T8894: `differential.generated-paths` doesn't actually prevent code from being shown in Differential.

That's how I have the value set and it still expands by default files whose
path matches one of those regexes.

Jul 24 2015, 6:33 AMDifferential
nornagon added a comment to T8659: Allow linking to an external build system from Harbormaster.

@dpaola2 at this point I don't even remember all the things I had set up, but it involves this monstrosity running on heroku:

1var request = require('request');
2var express = require('express');
3var bodyParser = require('body-parser');
4var app = express();
5var createCanduit = require('canduit');
6
7app.set('port', (process.env.PORT || 5000));
8app.use(bodyParser.json())
9
10app.post('/build', function(req, res) {
11 var target_phid = req.query.target;
12 var diff_id = req.query.diff;
13 request.post({
14 url: 'https://circleci.com/api/v1/project/transcriptic/www/tree/circle',
15 qs: { "circle-token": process.env['CIRCLE_TOKEN'] },
16 json: true,
17 body: {
18 "build_parameters": {
19 "PHAB_DIFF_ID": diff_id,
20 "PHAB_TARGET_PHID": target_phid
21 }
22 }
23 }, function (err, res, body) {
24 console.log(err, body)
25 })
26 res.end();
27});
28
29app.post('/build-complete', function(req, res) {
30 var target_phid = req.body.payload.build_parameters['PHAB_TARGET_PHID'];
31 var outcome = req.body.payload.outcome
32
33 createCanduit({
34 api: "https://work.r23s.net/api/",
35 user: process.env['CONDUIT_USER'],
36 cert: process.env['CONDUIT_CERT'],
37 }, function(err, canduit) {
38 if (err) {
39 res.end();
40 return console.error("Error authenticating with conduit: %j", err);
41 }
42 canduit.exec('harbormaster.sendmessage', {
43 buildTargetPHID: target_phid,
44 type: (outcome == "success" ? "pass" : "fail"),
45 }, function(err, data) {
46 console.log("Called conduit", err, data)
47 res.end();
48 })
49 })
50})
51
52app.listen(app.get('port'), function() {
53 console.log('Node app is running on port', app.get('port'));
54});

Jul 24 2015, 6:20 AMPrioritized, Harbormaster
nornagon edited P1832 circle-phab-glue.js.
Jul 24 2015, 6:19 AM
nornagon reopened T8894: `differential.generated-paths` doesn't actually prevent code from being shown in Differential as "Open".

It seems to show the "Current Value" as JSON in the config summary, but it's set to a newline-separated list in the actual value:

Jul 24 2015, 6:17 AMDifferential

Jul 19 2015

nornagon created T8894: `differential.generated-paths` doesn't actually prevent code from being shown in Differential.
Jul 19 2015, 4:13 AMDifferential

Jun 24 2015

nornagon created T8659: Allow linking to an external build system from Harbormaster.
Jun 24 2015, 7:37 PMPrioritized, Harbormaster
nornagon awarded D13313: Email Preferences - round out options for Projects and Phriction a Doubloon token.
Jun 24 2015, 5:34 PM

Jun 16 2015

matelich awarded T8550: Email preference for Phriction document "changed subscribers" a Like token.
Jun 16 2015, 12:27 PMPhriction

Jun 15 2015

nornagon created T8550: Email preference for Phriction document "changed subscribers".
Jun 15 2015, 4:11 PMPhriction

Jun 11 2015

ox awarded Changelog a Yellow Medal token.
Jun 11 2015, 4:03 PM

May 29 2015

nornagon added a comment to T8359: Switching between images in Pholio causes a full reload of the image.

It looks like it tries to preload (I see requests in the network tab in devtools for all the images in the mock as soon as I load the page), but doesn't do anything to keep the preloaded data around.

May 29 2015, 9:38 PMPholio
nornagon added a comment to T8359: Switching between images in Pholio causes a full reload of the image.

Possibly also relevant: I don't have security.alternate-file-domain set. Is that required to have cacheable images?

May 29 2015, 7:32 PMPholio
nornagon renamed T8359: Switching between images in Pholio causes a full reload of the image from Switching between images in Pholio should be fast to Switching between images in Pholio causes a full reload of the image.
May 29 2015, 7:32 PMPholio
nornagon added a comment to T8359: Switching between images in Pholio causes a full reload of the image.

Also: the file appears to have 'Cacheable: no':

May 29 2015, 7:29 PMPholio
nornagon added a comment to T8359: Switching between images in Pholio causes a full reload of the image.

Potentially relevant: I see pholio preloading the images on pageload, but the server seems to be returning Cache-Control: no-cache, which might be what's causing Chrome to reload the image.

May 29 2015, 7:19 PMPholio
nornagon created T8359: Switching between images in Pholio causes a full reload of the image.
May 29 2015, 7:10 PMPholio

Mar 19 2015

nornagon added a comment to T7626: Attaching files to a Phame post doesn't mark them as public.

(In fact, there's no attached tab even present)

Mar 19 2015, 7:20 PMPhame
nornagon added a comment to T7626: Attaching files to a Phame post doesn't mark them as public.

If it helps, the phame post doesn't show up under the 'attached' tab on the
file.

Mar 19 2015, 7:18 PMPhame
nornagon created T7626: Attaching files to a Phame post doesn't mark them as public.
Mar 19 2015, 6:13 PMPhame

Mar 12 2015

nornagon created T7544: Submitting a comment on a Ponder answer doesn't update the UI.
Mar 12 2015, 9:57 PMPonder

Feb 9 2015

nornagon added a comment to T929: Allow two arbitrary revisions to be compared in Diffusion.

It would also be great to be able to see all the commits in a range, e.g. master@{1week}...master. (Comes in handy when writing changelogs.)

Feb 9 2015, 11:00 PMDiffusion (v3), Restricted Project

Feb 4 2015

nornagon added a comment to T7162: Consider CodeMirror or similar for leaving comments on code reviews.

Gotcha, yeah I would definitely shy away from trying to rebuild something like CodeMirror.

Feb 4 2015, 6:48 PMDifferential
nornagon created T7162: Consider CodeMirror or similar for leaving comments on code reviews.
Feb 4 2015, 4:54 PMDifferential

Dec 17 2014

nornagon renamed T6770: Clicking "Show older changes" in Pholio produces PhabricatorDataNotAttachedException from Clicking "Show older changes" in Pholio produces an error to Clicking "Show older changes" in Pholio produces PhabricatorDataNotAttachedException.
Dec 17 2014, 2:46 AMTransactions, Pholio
nornagon created T6770: Clicking "Show older changes" in Pholio produces PhabricatorDataNotAttachedException.
Dec 17 2014, 2:46 AMTransactions, Pholio

Nov 5 2014

nornagon created T6478: Customizable tokens.
Nov 5 2014, 6:14 PM

Oct 31 2014

nornagon added a comment to T6440: Green background on syntax highlighted code for char-level changes has weird padding.

Chrome on OS X.

Oct 31 2014, 5:29 PM
nornagon created T6440: Green background on syntax highlighted code for char-level changes has weird padding.
Oct 31 2014, 5:09 PM

Oct 27 2014

nornagon updated the task description for T6402: {Fxxx, layout=inline} doesn't actually layout inline with an image.
Oct 27 2014, 7:18 PMRemarkup
nornagon created T6402: {Fxxx, layout=inline} doesn't actually layout inline with an image.
Oct 27 2014, 7:17 PMRemarkup

Aug 5 2014

nornagon created T5797: Bold/italic/underline doesn't work inside links.
Aug 5 2014, 4:22 PM

Jul 31 2014

nornagon created T5758: Package auditing should only require auditing on closing branches.
Jul 31 2014, 3:59 PMRestricted Project, Owners

Jul 25 2014

nornagon added a comment to T5705: "Time spent" in Phrequent is incorrect.

It definitely worked before rP2101c3b, when I used the 'Start Tracking Time' and 'Stop Tracking Time' buttons on the web interface. Unfortunately that revision doesn't revert cleanly, so I wasn't able to test easily.

Jul 25 2014, 4:54 PMPhrequent
nornagon created T5705: "Time spent" in Phrequent is incorrect.
Jul 25 2014, 12:41 AMPhrequent

Jul 19 2014

nornagon created T5656: Add "Create Similar Rule..." to Herald.
Jul 19 2014, 12:19 AMRestricted Project, Herald

Jun 6 2014

nornagon created T5280: Feature request: search for tasks assigned to me which don't have a Differential revision attached.
Jun 6 2014, 8:21 PMManiphest