Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Remove duplicate river comments (Trac #4925) #4925

Closed
elgg-gitbot opened this issue Feb 16, 2013 · 15 comments
Closed

Remove duplicate river comments (Trac #4925) #4925

elgg-gitbot opened this issue Feb 16, 2013 · 15 comments

Comments

@elgg-gitbot
Copy link

Original ticket http://trac.elgg.org/ticket/4925 on 42894121-10-12 by trac user mrclay, assigned to unknown.

Elgg version: 1.8

When an object is new, and contains comments, users may see duplicate comments: some attached to the object's create item, and also individual river items for these.

Before view rendering, we could remove comment items from the river that are already displayed underneath an object. This would make some pages have fewer items that others, but this seems acceptable.

@elgg-gitbot
Copy link
Author

trac user rivervanrain wrote on 42895118-05-12

Commit: [https://github.com/RiverVanRain/Elgg/commit/f8359f983877aa5971497bbcf5f1fe87c35ed939]

@elgg-gitbot
Copy link
Author

trac user mrclay wrote on 42895638-09-04

rivervanrain but once the object creation item drops off the first page, you'll no longer see new comments at the top of the stream. You need both river items.

If someone posts a new comment, I expect that at the top of the stream, not 3 pages down just because the item commented on is old.

@elgg-gitbot
Copy link
Author

trac user webgalli wrote on 42916446-07-20

What about an extra column for last_action and list by last_action?

@ewinslow
Copy link
Contributor

This continues to be a frustrating problem....

@mrclay
Copy link
Member

mrclay commented Sep 11, 2014

My main project bumps the parent object river item timestamp on receiving a comment and only shows individual comment river items when viewing a single user's stream. This makes the river bounce a lot but still seems more usable than showing comments out of context.

I think I could reduce the bouncing by only bumping when the latest comment is more than N seconds after the last bump time.

@mrclay
Copy link
Member

mrclay commented Sep 11, 2014

...then highlight comment(s) since the previous bump.

@dranii
Copy link

dranii commented Dec 24, 2014

"My main project bumps the parent object river item timestamp on receiving a comment and only shows individual comment river items when viewing a single user's stream. This makes the river bounce a lot but still seems more usable than showing comments out of context."

This is really good - how we can do this in Elgg 1.9.x/ Elgg 1.10 ?

@ewinslow
Copy link
Contributor

@dranii, new features go into 1.11 at this point.

And I really age this could improve things.

@ewinslow
Copy link
Contributor

agree*

@jdalsem jdalsem added this to the Elgg 4.0.x milestone Sep 26, 2019
jeabakker added a commit to jeabakker/Elgg that referenced this issue Mar 31, 2020
@jeabakker
Copy link
Member

We changed the all activity listing to not show comments/responses.
So the create blog river activity no longer shows the responses / comments. This way there is no duplication in the listing.
On the activity listing of the user the responses are shown

@jdalsem
Copy link
Member

jdalsem commented Sep 10, 2021

We changed the all activity listing to not show comments/responses.
So the create blog river activity no longer shows the responses / comments. This way there is no duplication in the listing.
On the activity listing of the user the responses are shown

This option does not help us... need to go for an option where we maybe do not create comment river items but instead bubble related river items (like blog create) to the top (so the responses are visible with their recent responses).. This might require an extra 'time_updated' timestamp with the river item

@jdalsem jdalsem reopened this Sep 10, 2021
@jdalsem jdalsem modified the milestones: Elgg 4.0 - beta, Elgg 4.1 Sep 10, 2021
@jdalsem jdalsem modified the milestones: Elgg 4.1, Elgg 4.2 Nov 24, 2021
@iionly
Copy link
Contributor

iionly commented Nov 24, 2021

How about that: strictly speaking, there are no duplicate river entries!

The impression of duplicate river entries only comes from the extra "service" of showing the latest comments posted to an entity in the river to allow for responding directly from the activity page. Only in the case that there haven't been any other river entries added between the entity added and the comments made it might seem that there are duplicate river entries. But there are not the same!

No longer added a separate river entry for the comment but only update the time_updated for the entity added river entry seems not a good idea because you will only see the latest 3 comments made on the activity page but no longer any other older comments. And it breaks the chronological order of the river entries!

How about an alternative way to deal with it: leave the separate comment entries in the river unchanged. But adding a site option to set a default regarding the presentation of the comments in the original entity river entry. The option would allow for showing the responses either like they are now (for people not caring about the "duplicate" comments) or alternatively show the responses collapsed and only make them visible on demand.

@daniyaree
Copy link

I think the way things were in Elgg 3x was quite good and perhaps also in 4x - need to test.
I fully like this:
"The impression of duplicate river entries only comes from the extra "service" of showing the latest comments posted to an entity in the river to allow for responding directly from the activity page. Only in the case that there haven't been any other river entries added between the entity added and the comments made it might seem that there are duplicate river entries. But there are not the same!"
I am not sure I have understood the 3rd part (an alternative way) - some screenshots of the suggestion would be good, if there is time. I would like to test this more for myself and post what I found/what I feel - do we have others also interested in this, as this is a vital way how we look at the site on first entry after logging in. Thanks @iionly

@jdalsem jdalsem modified the milestones: Elgg 4.2, Elgg 4.3 Mar 15, 2022
@jdalsem
Copy link
Member

jdalsem commented Jul 7, 2022

This needs more thinking and probably is going to be breaking. Putting this on Elgg 5 for now.

@jdalsem jdalsem modified the milestones: Elgg 4.3, Elgg 5 Jul 7, 2022
jdalsem added a commit to jdalsem/Elgg that referenced this issue Nov 16, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Development

No branches or pull requests

8 participants