View Issue Details

IDProjectCategoryView StatusLast Update
0022933mantisbttimelinepublic2017-06-04 14:58
Reporteratrol Assigned Toatrol  
Status closedResolutionfixed 
Target Version2.5.0Fixed in Version2.5.0 
Summary0022933: Confusing entry in timeline when removing other users from monitoring list

User A adds himself to monitoring list.
Timeline shows: User A is monitoring issue: OK

User A adds user B to monitoring list.
There is no entry in timeline: Questionable, but OK as this has been implemented to prevent entries when sending reminders

User A removes user B from monitoring list.
Timeline shows: User A stopped monitoring issue: NOK

TagsNo tags attached.




2017-05-27 14:04

developer   ~0056972




2017-05-28 19:17

manager   ~0056980

@atrol It is unclear to me why all these events shouldn't show on the timeline. I would expect to see all of the 3 cases.



2017-05-30 14:04

developer   ~0056992

@vboctor I just implemented the same logic for unmonitoring which is already implemented for monitoring.
This helps at least to get rid of the confusing message.

We would need additional messages if we want a correct display of such events in timeline.
User A added user B to monitoring list of issue #<IssueId>
User A removed user B from monitoring list of issue #<IssueId>

There are some more activities which get not listed in Timeline at the moment, e.g. adding / removing relationships, adding / removing attachments.
The general question is: Do we want see all activity in Timeline or wouldn't this be too much of information?
We would need some kind of configurable filter in UI for it.

This goes beyond the scope of this PR and should be tracked as a separate feature request.

For the moment, I recommend to merge the PR as is.



2017-05-31 23:24

manager   ~0057008

Got the context and approved the PR.

I think we should reflect all the actions that change state of issues in the timeline. We could even increase the scope for other entities in the future (e.g. projects, categories, versions, etc -- though we don't record such information so I wouldn't worry about them).

Related Changesets

MantisBT: master a864b24f

2017-05-27 08:03


Details Diff
Don't show unmonitoring of other users in timeline

Fixes 0022933
Affected Issues
mod - core/timeline_api.php Diff File