mahara-contributors team mailing list archive

[Mahara Bot <1873119@xxxxxxxxxxxxxxxxxx>]

Reviewed:  https://reviews.mahara.org/11970
Committed: https://git.mahara.org/mahara/mahara/commit/ea4fbaf21621130fe3ba75d56e4a484947a95852
Submitter: Robert Lyon (robertl@xxxxxxxxxxxxxxx)
Branch:    master

commit ea4fbaf21621130fe3ba75d56e4a484947a95852
Author: Doris Tam <doristam@xxxxxxxxxxxxxxx>
Date:   Tue Aug 31 21:40:05 2021 +1200

Bug 1873119: Exports: connect comments block to export config

Comment blocks will also be considered on portfolio exports

Change-Id: I50256ebf6b46069aa8181699a457daabc99700fb

-- 
You received this bug notification because you are a member of Mahara
Contributors, which is subscribed to Mahara.
Matching subscriptions: mahara-contributors
https://bugs.launchpad.net/bugs/1873119

Title:
  Artefact + general comments are they appearing in HTML / PDF exports

Status in Mahara:
  Fix Committed

Bug description:
  If they are appearing in HTML export in a modal then we need to get
  them to appear in PDF a different way (probably appended to the page
  comments)

  The HTML export area for comments is out of date, also which is used
  by the PDF also. The formatting needs to be updated.

  Additional to this, adding a switch to allow private-comment-free
  exports.

  To test this:

  1. Create a page with multiple blocks
  2. Make comments to the artefacts displaying in the blocks - using a combination of the 'details' view as well as adding private and public comments
  3. Export the page with the 'Include comments' ON and 'Include private comments OFF'
     - ✅ CHECK when 'Include comments' is OFF, 'Include private comments' switch is disabled
  4. ✅ EXPECT - none of the private comments in the PDF export at the base page where the page comments sit.
  6. ✅ EXPECT - the artefacts comments do not appear on the HTML export and only the PDF export 
  7. Export the page again with 'Include comments' ON and 'Include private comments' ON
  8. ✅ EXPECT - the private comments appear in the PDF export with a  grey background

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