Page MenuHomePhabricator

ox (Artem Titoulenko)
Software Engineer at Spring

Projects

User does not belong to any projects.

Today

  • Clear sailing ahead.

Tomorrow

  • Clear sailing ahead.

Thursday

  • Clear sailing ahead.

User Details

User Since
Dec 8 2014, 9:09 PM (484 w, 13 h)
Availability
Available

Recent Activity

Nov 28 2017

ox added a comment to T8918: Header shows number of notifications and various other controls on the 2FA auth screen.

Thank you very much!

Nov 28 2017, 6:11 PM · Security, Auth

Dec 17 2015

ox added a comment to T10000: Consider restoring Clowncopterize.

The "Clowncopterize" button was a huge roadblock to adopting code reviews in Differential in our organization. The engineers refused to press it, opting instead of just email each other in the email Phabricator sent out when the Diff was created. Everyone was confused about what it did, and were too conservative to take the risk. We had to hire a PHP outsourcing company in Indiana to patch our deploy so the button instead displayed "Comment".

Dec 17 2015, 9:45 PM · Differential, Feature Request

Dec 15 2015

ox added a comment to T9905: Upgrading: ApplicationEditor changes in Maniphest.

Is there an expected date when this change will be safe for production use? That red warning bar at the top of the summary is very intimidating, though I want to keep our instance responsibly close to the edge (should I just use stable?).

Dec 15 2015, 11:25 PM · Maniphest, Installing & Upgrading

Nov 15 2015

ox added a comment to T9786: Creating Hosted Repo leads to Almanac Exception.

I definitely get that it's a prototype and I know not to file bugs against prototypes, but this interaction wasn't directly with Almanac but with a core Diffusion feature. Diffusion shouldn't use Almanac unless explicitly set to use it. 

Nov 15 2015, 12:54 AM

Nov 14 2015

ox created T9786: Creating Hosted Repo leads to Almanac Exception.
Nov 14 2015, 10:02 PM

Nov 4 2015

ox added a comment to T9711: Feature Request: Notifications about comments on Diffs.

Yea, I figured it was more on us. Their reason is:

Nov 4 2015, 5:36 PM · Differential
ox added a comment to T9711: Feature Request: Notifications about comments on Diffs.

Boss-person insists on auto-emailing all of eng for every bit of activity on our install, so we've had to create a ton of email filters. However every so often mail that should've passed gets scrapped, and it's different for every person. Some people cannot deal with the deluge of email and just direct all phab email to the trash.

Nov 4 2015, 5:19 PM · Differential
ox created T9711: Feature Request: Notifications about comments on Diffs.
Nov 4 2015, 4:48 PM · Differential

Sep 1 2015

ox created T9312: URL Param to pass to Paste to download paste instead of viewing it.
Sep 1 2015, 7:07 PM · Paste

Aug 21 2015

ox added Q101: 'Cannot allocate memory' when viewing large diffs (Answer 122).
Aug 21 2015, 1:45 PM

Aug 3 2015

ox abandoned D13759: Remove extra `/phurl` in request url.
Aug 3 2015, 6:28 PM

Jul 30 2015

ox added a comment to T9022: Phurl missing `PhabricatorPhurlCommentController` and commenting route.

Oh! I didn't notice the prototype tag all the way on the right (we have really wide monitors). That is totally fair. Apologies for wasting your time on prototype bugs.

Jul 30 2015, 2:14 PM · Phurl
ox added a comment to T9022: Phurl missing `PhabricatorPhurlCommentController` and commenting route.

@chad: Not even eventually?

Jul 30 2015, 2:09 PM · Phurl
ox added a comment to D13759: Remove extra `/phurl` in request url.

I don't know enough PHP to really get what this app is doing, but I imagined some smart folks at Phacility had created a generic commenting service and I was just not seeing it. Thanks for the help!

Jul 30 2015, 2:07 PM
ox created T9022: Phurl missing `PhabricatorPhurlCommentController` and commenting route.
Jul 30 2015, 2:04 PM · Phurl
ox updated the diff for D13759: Remove extra `/phurl` in request url.

remove leading / as it was getting trimmed anyway

Jul 30 2015, 3:16 AM
ox retitled D13759: Remove extra `/phurl` in request url from to Remove extra `/phurl` in request url.
Jul 30 2015, 12:15 AM
ox added a revision to T9009: Cannot comment on Phurl links: D13759: Remove extra `/phurl` in request url.
Jul 30 2015, 12:15 AM · Phurl

Jul 29 2015

ox added a comment to T9009: Cannot comment on Phurl links.

yup: https://secure.phabricator.com/diffusion/P/browse/master/src/applications/phurl/controller/PhabricatorPhurlURLViewController.php;b24935df2f74006c8221ee934c3d06b0781ac70b$48

Jul 29 2015, 11:21 PM · Phurl
ox created T9009: Cannot comment on Phurl links.
Jul 29 2015, 11:18 PM · Phurl

Jul 27 2015

ox created T8975: Reference variable for the current user on Conduit Method Call page.
Jul 27 2015, 9:47 PM · Conduit

Jul 22 2015

ox added a comment to T8918: Header shows number of notifications and various other controls on the 2FA auth screen.

Thankfully clicking the notifications opens a dropdown with "undefined" in it. Maybe tagging with Security was overzealous but every MFA screen I've ever encountered showed no data from the underlying system. At worst it just looks sloppy since half of the features don't return useful data, the custom logo disappears (it's not accessible, therefore doesn't return), etc. Also, like @Lavoaster points out, there could be info in saved searches. This isn't huge, just something that should be fixed eventually.

Jul 22 2015, 3:36 AM · Security, Auth

Jul 21 2015

ox renamed T8918: Header shows number of notifications and various other controls on the 2FA auth screen from Header shows number of notifications and various other controls on the 2FA screen to Header shows number of notifications and various other controls on the 2FA auth screen.
Jul 21 2015, 6:20 PM · Security, Auth
ox created T8918: Header shows number of notifications and various other controls on the 2FA auth screen.
Jul 21 2015, 6:19 PM · Security, Auth

Jun 11 2015

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

Apr 21 2015

ox closed T7845: ArcanistGoTestResultParser incorrectly parses go test results for Go 1.4 as Resolved.
Apr 21 2015, 7:09 PM · Arcanist
ox requested review of D12436: allow either (<time> seconds) or (<time>s) format for test result time.
Apr 21 2015, 7:01 PM
ox added a comment to D12436: allow either (<time> seconds) or (<time>s) format for test result time.

strangely enough that config is already there. Maybe it's a linter bug?

Apr 21 2015, 7:01 PM
ox raised the priority of T7845: ArcanistGoTestResultParser incorrectly parses go test results for Go 1.4 from Low to Normal.
Apr 21 2015, 6:39 PM · Arcanist

Apr 20 2015

ox added a comment to D12436: allow either (<time> seconds) or (<time>s) format for test result time.

Better, @avivey?

Apr 20 2015, 8:50 PM

Apr 17 2015

ox updated the diff for D12436: allow either (<time> seconds) or (<time>s) format for test result time.
  • add tests to check go test parser against go 1.4 test result format
Apr 17 2015, 8:05 PM
ox added a revision to T7845: ArcanistGoTestResultParser incorrectly parses go test results for Go 1.4: D12436: allow either (<time> seconds) or (<time>s) format for test result time.
Apr 17 2015, 6:24 PM · Arcanist
ox added a task to D12436: allow either (<time> seconds) or (<time>s) format for test result time: T7845: ArcanistGoTestResultParser incorrectly parses go test results for Go 1.4.
Apr 17 2015, 6:24 PM
ox created T7845: ArcanistGoTestResultParser incorrectly parses go test results for Go 1.4.
Apr 17 2015, 6:23 PM · Arcanist

Apr 16 2015

ox awarded I2 Donate to Phacility a Mountain of Wealth token.
Apr 16 2015, 6:53 PM · Phacility
ox retitled D12436: allow either (<time> seconds) or (<time>s) format for test result time from to allow either (<time> seconds) or (<time>s) format for test result time.
Apr 16 2015, 4:25 PM