Page MenuHomePhabricator

Update tasks/commits, remove diffs from Profile
ClosedPublic

Authored by chad on Mar 6 2017, 5:58 PM.
Tags
None
Referenced Files
F18813362: D17470.diff
Mon, Oct 20, 3:40 PM
F18757338: D17470.id42012.diff
Sun, Oct 5, 4:58 PM
F18733492: D17470.id.diff
Tue, Sep 30, 10:35 PM
F18566103: D17470.id.diff
Sep 9 2025, 2:18 PM
F18500698: D17470.diff
Sep 4 2025, 9:15 PM
F18336437: D17470.diff
Aug 25 2025, 1:38 PM
F18094438: D17470.diff
Aug 7 2025, 10:07 PM
F18029688: D17470.id42015.diff
Aug 2 2025, 11:22 PM
Subscribers

Details

Summary

Mostly a minor nit-pick, but I hate sending users off the profile and disorient them onto application search. These pages are pretty easy to maintain, I don't expect to need to do more here. I dropped Differential outright. Kept Tasks and Commits. Now you can browse everything about a user on their profile without leaving. Maybe add a link to ApplicationSearch? Not sure it's important.

Test Plan

Review tasks and commits on mine and other user profiles.

Diff Detail

Repository
rP Phabricator
Branch
profile-tasks (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15918
Build 21078: Run Core Tests
Build 21077: arc lint + arc unit

Event Timeline

The value of keeping users on the profile seems lower to me than the value of having ApplicationSearch constraints to filter things, but I don't feel strongly.

This revision is now accepted and ready to land.Mar 6 2017, 6:10 PM

I never find these useful, offhand, but if I wanted to search for someone's tasks, I'd just outright use ApplicationSearch. At least here, if I'm on a Profile to learn about someone, I'm kept at the source.

This revision was automatically updated to reflect the committed changes.