Page MenuHomePhabricator

Remove timeline from Profile Manage
ClosedPublic

Authored by chad on Jan 19 2017, 8:57 PM.
Tags
None
Referenced Files
F15458693: D17230.diff
Mon, Mar 31, 6:27 AM
F15442578: D17230.id41442.diff
Wed, Mar 26, 11:49 PM
F15442577: D17230.id41441.diff
Wed, Mar 26, 11:48 PM
F15442576: D17230.id.diff
Wed, Mar 26, 11:48 PM
F15438159: D17230.diff
Tue, Mar 25, 11:37 PM
F15286896: D17230.id.diff
Tue, Mar 4, 7:30 PM
Unknown Object (File)
Feb 21 2025, 3:06 PM
Unknown Object (File)
Feb 14 2025, 11:03 PM
Subscribers

Details

Summary

Not sure this page is really providing any value, the timeline always says "edited this object" and there is a list of actions. Seems we could move actions back to the profile proper, but they feel very... engineery to me. Or we could fix the timeline stories, but my guess is they aren't useful or we would have gotten such feedback.

Test Plan

Review manage page, timeline is gone. Page is clean.

Diff Detail

Repository
rP Phabricator
Branch
prof-manage (branched from master)
Lint
Lint Passed
Unit
Tests Passed
Build Status
Buildable 15330
Build 20182: Run Core Tests
Build 20181: arc lint + arc unit

Event Timeline

I'd like to fix this timeline to show administrative events, particularly account disables/enables. This information is indirectly available in People > Activity Logs but inconvenient to access. However, I can put the timeline back after I fix it.

This revision is now accepted and ready to land.Jan 19 2017, 9:01 PM

I’m guessing that’s not something I could do?

This revision was automatically updated to reflect the committed changes.

You could, but it's enough work that I don't think it's worthwhile until we do a general update here (maybe related to generally improving logging/security stuff?), since "Disable / Enable" isn't transactional right now.

You could fix the feed as it exists today by adding rendering methods to the stuff in src/applications/people/customfield/, I think -- they're missing getApplicationTransactionTitle() methods, mostly. That would get us "epriestley changed titles from X to Y" instead of "epriestley edited this object.", at least.