Page MenuHomePhabricator

Update tasks/commits, remove diffs from Profile
ClosedPublic

Authored by chad on Mar 6 2017, 5:58 PM.
Tags
None
Referenced Files
F18903932: D17470.id.diff
Nov 8 2025, 6:57 AM
F18877721: D17470.diff
Nov 6 2025, 11:37 AM
F18843648: D17470.id42015.diff
Oct 28 2025, 11:56 PM
F18842146: D17470.diff
Oct 28 2025, 1:35 PM
F18838931: D17470.id42012.diff
Oct 27 2025, 1:31 PM
F18813362: D17470.diff
Oct 20 2025, 3:40 PM
F18757338: D17470.id42012.diff
Oct 5 2025, 4:58 PM
F18733492: D17470.id.diff
Sep 30 2025, 10:35 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.