ratcreature: RatCreature's toon avatar (Default)
RatCreature ([personal profile] ratcreature) wrote in [site community profile] dw_suggestions2011-01-03 11:11 pm

edited comment notifications in the DW inbox

Title:
edited comment notifications in the DW inbox

Area:
comments

Summary:
When a comment is edited the original notification in the inbox changes and also shows the edited version, but you still get a new notification with the edited comment. So basically you get duplicates with no value. I think that should be changed, so that only one notification with the edited comment is there.

Description:
With the email version of the notifications it makes sense that you get two, because one shows the old comment, the second the edited one, so you can see the edit. But in the DW inbox the notification reflects the current state of the comment (or entry), so the first notification itself is edited and changed to say "edited comment by" exactly the same as the second one.

So if the first is already updated the second is superfluous clutter. I don't think there would be any drawbacks to not having two of the same.

Poll #5578 edited comment notifications in the DW inbox
Open to: Registered Users, detailed results viewable to: All, participants: 57


This suggestion:

View Answers

Should be implemented as-is.
22 (38.6%)

Should be implemented with changes. (please comment)
6 (10.5%)

Shouldn't be implemented.
6 (10.5%)

(I have no opinion)
22 (38.6%)

(Other: please comment)
1 (1.8%)

thorfinn: <user name="seedy_girl"> and <user name="thorfinn"> (Default)

[personal profile] thorfinn 2011-01-09 02:01 pm (UTC)(link)
Perhaps the old notification should be unchanged in the inbox?
denise: Image: Me, facing away from camera, on top of the Castel Sant'Angelo in Rome (Default)

[staff profile] denise 2011-01-09 03:27 pm (UTC)(link)
Yeah, it's not possible, alas!
thorfinn: <user name="seedy_girl"> and <user name="thorfinn"> (Default)

[personal profile] thorfinn 2011-01-10 07:11 am (UTC)(link)
*ponder* Maybe the old notification could be modified to contain a "diff" (just an indicator of what changed), whist the latest one points to the latest actual content?
denise: Image: Me, facing away from camera, on top of the Castel Sant'Angelo in Rome (Default)

[staff profile] denise 2011-01-09 03:27 pm (UTC)(link)
Unfortunately that is not possible! (Inbox displays the comment object. Once comment object is edited, old version is written over.)
rebelsheart: Original Concept  by Me (Default)

[personal profile] rebelsheart 2011-01-09 05:09 pm (UTC)(link)
and now that I know more about databases thanks to my new job, i understand why!
thorfinn: <user name="seedy_girl"> and <user name="thorfinn"> (Default)

[personal profile] thorfinn 2011-01-10 07:16 am (UTC)(link)
Convert old notifications to diffs? Or store a comment diff separately and pointer to that instead?

Diff history could be useful for entry edits also.

Impact, of course, is database space required, retention periods, usefulness of diffs, increased complexity, etc. :-)

Benefit is this sort of thing, plus *much* better audit-trail if necessary, potentially a good way to store drafts and edits, etc.

If you store the diffs as reverse diff from the current state, it makes it easy to expire old/too-many diffs (i.e., keep only the last 5, or 10 or something).