Page MenuHomePhabricator

Remove timeline from Profile Manage
ClosedPublic

Authored by chad on Jan 19 2017, 8:57 PM.
Tags
None
Referenced Files
Unknown Object (File)
Tue, Nov 19, 1:13 PM
Unknown Object (File)
Mon, Nov 4, 10:52 PM
Unknown Object (File)
Oct 22 2024, 8:18 PM
Unknown Object (File)
Oct 21 2024, 10:00 AM
Unknown Object (File)
Oct 17 2024, 12:38 AM
Unknown Object (File)
Oct 9 2024, 9:57 AM
Unknown Object (File)
Oct 9 2024, 7:07 AM
Unknown Object (File)
Oct 9 2024, 5:51 AM
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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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.