Skip to content

6.39.1

Compare
Choose a tag to compare
@NGPixel NGPixel released this 16 Mar 22:45
· 7998 commits to main since this release

Summary: Historical review data. Enhanced person profile page. Review page enhancements.
Release Date: Fri, December 9, 2016 at 12:56 PM UTC
Release Author: Henrik Levkowetz


This is a mixed bugfix and feature release. From the commitlog:

  • Merged in [12489] from rjsparks@nostrum.com:
    Save a new reviewrequest for each team requested in the request form.
    Fixes #2094.

  • Merged in [12487] from rjsparks@nostrum.com:
    Make it clearer what version was requested and what version was
    reviewed on a reveiwers overview page. Fixes #2101.

  • Check event type only when an event is available.

  • Merged in [12482] from rjsparks@nostrum.com:
    Improves control of email headers for review summary messages. Provides
    team-specific templates for review summary messages. Fixes #2092 and #2082.

  • Merged in [12479] from olau@iola.dk:
    Add missing team constraint in the check for previous reviews by a
    reviewer.

  • Moved optional text wrapping before html escaping in markup_unicode(),
    used by get_unicode_document_content(). Fixes a problem with some
    document lines being wrapped when they should not be.

  • Fixed the manage assigned reviews template to handle the case when
    reviewed rev is blank. Fixes issue #2086.

  • Changed to using TestCase from ietf.utils in some places, for consistent
    test-suite failure messages.

  • Updated installation instructions. Run migrations as the web server user;
    migrations may create files which the datatracker should be able to write.

  • Don't count the template loading done in TemplateChecksTestCase towards
    template coverage numbers. Updated coverage numbers to match.

  • Merged in [12461] from rjsparks@nostrum.com:
    Added a migration to fetch text from reviews in the mail archives and
    populate the review documents. Fixes #2064. Will patch into production.

  • Added more document information to the personal profile pages at
    /person/$FULL_NAME, after a suggestion from Benoit Claise. Fixes issue
    #2066.

  • Added new display columns to the reviewer settings admin page.

  • Merged in [12438] from rjsparks@nostrum.com:
    Don't show teams with only No response reviews in the search result
    rows, using the same logic as used for the document main page. Fixes #2070.

  • Fixed a typo. Fixes issue #2073.

  • Added a validator for the reviewer settings' Filter Regexp field, to
    make sure that we don't later hit exceptions when trying to compile and use
    the regexp entered.

  • Tweaked the admin for doc.models.Document to capture a comment about
    changes made, and save those in the document history, using
    .save_with_history(). This makes the admin useful again for doing document
    changes. See issue #2067.

  • Changed the handling of DocEvent rev fields in document history. We
    should give DocEvent a rev field which is always filled in; the current
    handling is error prone and much to complicated.

  • Fixed errors in the test_interim_request_series() test.

  • Added a link to the meetecho session recording (not the youtube
    conversion) on the IETF agenda page.

  • Added guards against picking up non-ascii characters in draft title and
    abstract from drafts submitted in xml form.


Coverage

chart