View Issue Details

IDProjectCategoryView StatusLast Update
2904Composrcns_forumpublic2016-10-28 14:54
ReporterPDStig Assigned To 
PrioritynormalSeverityfeature 
Status newResolutionopen 
Summary2904: Change detection for forum posts
DescriptionImplement change_detection.php for forum posts and comments so that users can see new posts as they are made.

I can commit this myself if accepted. I've already implemented it on lovinity.org.
Additional InformationMay also want to implement it on the full editor if possible where the posts display below the editor.
TagsNo tags attached.
Attach Tags
Time estimation (hours)16
Sponsorship open

Sponsor

Date Added Member Amount Sponsored

Relationships

related to 2903 ResolvedSalman Change detection refresh improvement 
related to 2905 ResolvedChris Graham Show message on change detection refresh 
related to 2906 ClosedChris Graham No automatic refresh on change detection; let user decide 
related to 3660 Not Assigned AJAXified posting 

Activities

Chris Graham

2016-10-28 14:53

administrator   ~4493

I suspect this is more complex than you think to do well enough for mainstream Composr...

1) We would need to have a "has not changed" signal that is extremely efficient, preferably before any database connection is opened. This is how the AJAX checks for notifications and chat messages work, because people will leave things open a long time so the regular checks have to be very fast to process.

2) If pagination is active and you're not on the last page, it shouldn't show new posts (presumably).

3) If threaded view is active we definitely shouldn't refresh the whole topic, so we need to load up individual posts properly, and insert them in the right positions in the tree.

Previously I was also concerned about adding table rows dynamically without making deep assumptions about templates, but with modern browsers this should not be an issue, and we moved to divs anyway.

Chris Graham

2016-10-28 14:54

administrator   ~4494

Oh, and it needs to be configurable. Many webhosts prohibit polling.

Add Note

View Status
Note
Upload Files
Maximum size: 32,768 KiB

Attach files by dragging & dropping, selecting or pasting them.
You are not logged in You are not logged in. This means you will not get any e-mail notifications. And if you reply, we will not know for sure you are the original poster of the issue.

Issue History

Date Modified Username Field Change
2016-10-27 21:28 PDStig New Issue
2016-10-27 21:28 PDStig Additional Information Updated
2016-10-27 21:29 PDStig Description Updated
2016-10-27 21:30 PDStig Relationship added related to 2903
2016-10-27 21:32 PDStig Relationship added related to 2905
2016-10-27 21:38 PDStig Relationship added related to 2906
2016-10-28 14:53 Chris Graham Time estimation (hours) => 16
2016-10-28 14:53 Chris Graham Note Added: 0004493
2016-10-28 14:54 Chris Graham Note Added: 0004494
2018-08-05 13:41 Chris Graham Relationship added related to 3660