You are here: Foswiki>Tasks Web>Item9459 (06 Oct 2011, GeorgeClark)Edit Attach

Item9459: Need diff mode for MailerContrib

pencil
Priority: Enhancement
Current State: Proposal Required
Released In: n/a
Target Release: n/a
Applies To: Extension
Component: MailerContrib
Branches:
Reported By: VickiBrown
Waiting For:
Last Change By: GeorgeClark
MailerContrib can currently do one of two things:
  • send motification that a page has changed
  • Send the content of the page as HTML

A third option would be very useful -- Send email with the diff whenever someone changes a page.

That is, don't just tell me "this page changed, clink the link to see the page, then click something else to see the changes". Tell me what changed!

-- VickiBrown - 11 Aug 2010

This is an enhancement request. Not a bug where something does not work.

-- KennethLavrsen - 12 Aug 2010

I totally agree that this enhancement would be useful.

-- ArthurClemens - 12 Aug 2010

The summariseChanges function in 1.1 sends out a limited diff of the changes, not the complete change. Is this sufficient for this task? Please check out the notification e-mails coming from Foswiki.org since late yesterday. (The diff function was broken previously)

-- GeorgeClark - 29 Mar 2011

The current diff mode actually is limited to 70 characters per line and 6 lines total. So a feature request might be to make these limits (currently CONSTANTs in Foswiki::Meta ) controllable by additional parameters to Func, and either configurable in the preferences, or settable in the LocalSite.cfg. Preference variables would probably be more difficult since they are per web / user / topic while the emails are I believe generated for multiple users.

-- GeorgeClark - 29 Mar 2011

I just checked in some work in progress on a DIFF mode. It may prove useful.

-- CrawfordCurrie - 05 Apr 2011

Does any of this rise to the level of needing documentation in the 1.1.4 release notes? There are commits to the release branch.

-- GeorgeClark - 02 Oct 2011

"Improved difference presentation in change notification emails" should do it.

-- CrawfordCurrie - 03 Oct 2011

Added this task to the 1.1.4 release notes manually using your description, plus (partial fix). Leaving this task open for the real fix - set to Proposal Required.

-- GeorgeClark - 06 Oct 2011
 
Topic revision: r13 - 06 Oct 2011, GeorgeClark
The copyright of the content on this website is held by the contributing authors, except where stated elsewhere. See Copyright Statement. Creative Commons License    Legal Imprint    Privacy Policy