mahara-contributors team mailing list archive

[Pramith Dayananda <pramithkm@xxxxxxxxx>]

Hi,

With my testing I can confirm the emails are sent to the "Group admins".
This is done via the "lib/activity". Thought there's a comment in the
code close to activity occurred call, saying "TODO: need to notify the
group admins bug #1197197".

Let me know if this needs changing.

Thanks,
Pramith.

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: Subscription for all Mahara Contributors -- please ask on #mahara-dev or mahara.org forum before editing or unsubscribing it!
https://bugs.launchpad.net/bugs/1197197

Title:
  No comment modification notification for group view

Status in Mahara:
  Confirmed

Bug description:
  Following on from  https://bugs.launchpad.net/mahara/+bug/1171310

  The patch mentioned there does not allow for group administrators to
  be alerted when a comment is modified from private to public with
  activate comments & comment moderation turned on for the page

  To test:
  get patch from https://bugs.launchpad.net/mahara/+bug/1171310 (if not already merged into master)

  1. Create a Group and make a page for it - take note of the url for the page
  2. Make the page accessible to the public, and activate comments & comment moderation for the page (this is all under the Sharing tab)
  3. Log in as a user not in the group and go to the page
  4. Place a comment on the Page, making sure to untick the "Make public" box so that the comment is private.
  5. Click the "edit" icon next to the newly created comment.
  6. On the edit page, tick the "Make public" box, and click Save.

  Expected result: The comment's status should be "This comment is
  private | You have requested that this comment be made public" and
  that the administrator of the group gets an email.

  Outcome: no email to group administrator

To manage notifications about this bug go to:
https://bugs.launchpad.net/mahara/+bug/1197197/+subscriptions