1. 12 Feb, 2021 1 commit
  2. 11 Feb, 2021 1 commit
    • Evonne's avatar
      Bug 1912414 Style updates · 6c312125
      Evonne authored
      - Make comment public /portfolio/feedback.html: sits in the middle and
        doesn't have the correct class.
      - Comment with attachment: size is still displayed and download icon
        doesn't have tooltip that includes file name and size.
      - Objectionable content: When page seen in 'Shared by me' page, it's
        missing the 'Portfolio under review' markup.
      - Plans: The word 'Template' is too small on the plans overview page.
      - Plans: View page and also Edit: Remove red background for overdue plan
        item as well as the red from its heading.
      - Plans: Edit on a page: The chevron on the task sits awkwardly.
      - Mark a forum post as objectionable: The alert icon to the left of the
        profile icon of the post that you are reporting is missing.
      - Once forum post has been reported, change the red text for 'Reported
        details' to black.
      - Legal: Expanded item has the chevron in the wrong direction.
      - Legal: The 'Last update' date moved to the bottom from the top right
        hand corner.
      - Elasticsearch result page, journal entry is pushed to the left.
      - Account settings in admin area: Calendar icon below last login sits
        outside of box.
      - Skin: - heading within a text block uses the block heading font when
        it should not do that.
      - Group's about page - under Group Portfolios heading, when a page/
        collection is a template, it has a button on the right to allow
        copying, tpl needs to be updated so it aligns to top of the
        list-group-item.
      - Look into the btn-group-top responsiveness, too many overrides for
        specific pages, like comments remove button, it seems to require an
        extra pixel around md breakpoint, would like to clean this up.
      
      behatnotneeded
      
      Change-Id: Icfffed9376e89339a8fad5a207d3b974b431e451
      6c312125
  3. 04 Feb, 2021 3 commits
  4. 03 Feb, 2021 1 commit
  5. 02 Feb, 2021 5 commits
  6. 29 Jan, 2021 2 commits
  7. 28 Jan, 2021 1 commit
    • Gold's avatar
      Bug 1895066 · e105d8aa
      Gold authored
      Draft peer assessment needs additional indicator besides colour
      
      Added a 'draft' string to "view" and used that in the template.
      Updated the 'draft' string in "blocktype.peerassessment/peerassessment"
      to 'savedraft' to prevent confusion.
      Added 'pubstatus' to the item if it is private and set this to Draft.
      Added pubstatus to the template if set to private. Style for this
      replicates the private status on Bug 1912414.
      
      Change-Id: I9cb0b20f9e00536e3162c3f83583b4877a6913a9
      e105d8aa
  8. 25 Jan, 2021 1 commit
  9. 20 Jan, 2021 4 commits
  10. 18 Jan, 2021 1 commit
  11. 14 Jan, 2021 1 commit
  12. 13 Jan, 2021 1 commit
  13. 11 Jan, 2021 1 commit
  14. 23 Dec, 2020 3 commits
  15. 22 Dec, 2020 5 commits
  16. 21 Dec, 2020 3 commits
  17. 20 Dec, 2020 1 commit
  18. 19 Dec, 2020 1 commit
  19. 18 Dec, 2020 1 commit
    • Kevin Dibble's avatar
      Bug 1803101: peer assessment artefact import · 6d19103a
      Kevin Dibble authored
      Need to define owner and block when creating a
      peer assessment artefact while importing
      
      Create a temp block to assign on artefact creation
      that later needs to be replaced by the actual block
      when it's available
      
      Draft assessments should not be exported or imported.
      For exported portfolios created on older version of
      Mahara, there is no way of knowing if they are drafts
      at the moment of importing because this information
      was not included in the leap2A export xml file.
      Not importing draft peer assessments should work
      from this patch.
      
      It also adds a fix to support
      Postgres and MySQL when a table
      column name is a reserved word.
      
      Removing not null restriction for "usr" field in "artefact_peer_assessment"
      table:
      - The author in peer assessment table should be working as comments table.
      - For each comment, there is an entry in 'artefact' table that contains
      author and authorname, and another entry in 'artefact_comment_comment'
      that doesn't have an author field
      because we already have one in 'artefact' table.
      - We should have something similar on 'artefact_peer_assessment'.
      There is already an author field in 'artefact' table,
      then the 'usr' field in 'artefact_peer_assessment' should not to have a
      'not null' restriction. If the peer assessment artefact has been imported
      and the author is not present in this site, then the 'usr'
      should be allowed to be null.
      
      behatnotneeded
      
      Change-Id: I182520159d1fcbc072081c2d976cdc68cde5e19d
      6d19103a
  20. 17 Dec, 2020 1 commit
  21. 16 Dec, 2020 2 commits
    • Robert Lyon's avatar
    • Doris Tam's avatar
      Bug 1866779: moving watchlist delay notification settings · 5d55b90c
      Doris Tam authored
      Moving the watchlist delay notification setting from site options from admin settings
      to plugin administration under blocktype.
      
      - Moved strings to the blocktype.watchlist lang strings area.
      - Removed a set_config call in upgrade.php and added a post_inst method
      following other blocktype plugin  post install calls instead.
      
      - Upgrade step to transfer over old site setting 'watchnotification_delay'
      to plugin admin, updating the version number too
      Change-Id: If388ddd083ceb8f46026fdc138f27ca8c8da5b04
      5d55b90c