Commit graph

1653 commits

Author SHA1 Message Date
Robert Sparks f4fd4b1921 Gather actual repos to backup takinging user and organizational owners into account, iterating through owner repos when necessary using the github api.
- Legacy-Id: 18384
2020-08-19 19:07:29 +00:00
Robert Sparks 2b70735fd2 Improved the classification of some github related external resources. Tightened validation of new resource values. Commit ready to merge.
- Legacy-Id: 18378
2020-08-17 23:50:22 +00:00
Henrik Levkowetz 9125104065 Merged in [18324] from rjsparks@nostrum.com:
Send extra mail to yangdoctors chairs when a draft with yang checks goes into IETF LC. Fixes #2419.
 - Legacy-Id: 18363
Note: SVN reference [18324] has been migrated to Git commit cb58810705
2020-08-13 11:23:22 +00:00
Robert Sparks cf42db8804 Show link to a drafts group page for all group types when the draft has a group. Fixes #3061. Commit ready for merge.
- Legacy-Id: 18357
2020-08-12 18:47:29 +00:00
Henrik Levkowetz 57938b039d Converted many cases of plain-text 403 messages to use a properly styled page instead, with a login link when appropriate. Also changed some API endpoint 400 responses to a more correct 403.
- Legacy-Id: 18339
2020-08-06 10:59:52 +00:00
Robert Sparks cb58810705 Send extra mail to yangdoctors chairs when a draft with yang checks goes into IETF LC. Fixes #2419. Commit ready to merge.
- Legacy-Id: 18324
2020-07-30 21:44:02 +00:00
Henrik Levkowetz e4bc508abc Merged in [18298] from rjsparks@nostrum.com:
Convert IRTF area groups into their own group type rather than attempting to overload AG. Fixes #3027.
 - Legacy-Id: 18320
Note: SVN reference [18298] has been migrated to Git commit 2b10465aa7
2020-07-30 14:27:17 +00:00
Henrik Levkowetz 9dbdbd5078 Merged in [18296] from rjsparks@nostrum.com:
Show RSOC from the Group menu. Fixes #3024.
 - Legacy-Id: 18308
Note: SVN reference [18296] has been migrated to Git commit 328c92f395
2020-07-29 14:18:29 +00:00
Henrik Levkowetz 57f6fba930 Merged in [18276] from housley@vigilsec.com:
Send email to AD when an IETF Last Call expires that contains downrefs. Fixes #2472
 - Legacy-Id: 18306
Note: SVN reference [18276] has been migrated to Git commit e0ca07e65d
2020-07-29 13:08:10 +00:00
Robert Sparks 2b10465aa7 Convert IRTF area groups into their own group type rather than attempting to overload AG. Fixes #3027. Commit ready for merge.
- Legacy-Id: 18298
2020-07-28 18:54:44 +00:00
Robert Sparks 44d19f0533 Let MyPy know a decoration is on purpose. Commit ready for merge.
- Legacy-Id: 18297
2020-07-28 16:49:08 +00:00
Robert Sparks 328c92f395 Show RSOC from the Group menu. Fixes #3024. Commit ready for merge.
- Legacy-Id: 18296
2020-07-28 16:10:42 +00:00
Henrik Levkowetz f10255e0e2 Merged in [18270] from valery@smyslov.net:
Added display of verified errata in red, not verified in orange in document lists.

    #2997
 - Legacy-Id: 18280
Note: SVN reference [18270] has been migrated to Git commit 126d32b37f
2020-07-26 14:46:35 +00:00
Russ Housley e0ca07e65d Send email to AD when an IETF Last Call expires that contains downrefs. Fixes #2472
- Legacy-Id: 18276
2020-07-25 21:09:50 +00:00
Valery Smyslov 126d32b37f #2997
- Legacy-Id: 18270
2020-07-25 17:25:31 +00:00
Henrik Levkowetz d16e946ed3 Merged in [18189] from jennifer@painless-security.com:
Make docs in Auth48 state visually distinct. Fixes #2925.
 - Legacy-Id: 18210
Note: SVN reference [18189] has been migrated to Git commit abe97eeb28
2020-07-21 09:28:53 +00:00
Henrik Levkowetz d6d3d82434 Merged in [18198] from rjsparks@nostrum.com:
Return the right reference for interim meeting materials.
 - Legacy-Id: 18200
Note: SVN reference [18198] has been migrated to Git commit 9bcd64455a
2020-07-20 18:09:16 +00:00
Robert Sparks 9bcd64455a Return the right reference for interim meeting materials. Commit ready for merge.
- Legacy-Id: 18198
2020-07-20 17:39:33 +00:00
Henrik Levkowetz 606cedd796 Merged in the ExtResource work from rjsparks@nostrum.com, based on a feature request and contributions from rsalz@akamai.com. This provides external resource models for Person, Group, and Document models, as a generalisation of the previous related-URL concept. This provides a consistent labelling and classification of URLs and other resources related to a Group, Document, or Person. The GroupURL (and similar) classes will be removed in a later step. Some DocumentURL instances (such as auth48 URLs) remain to convert, as they have come in with other merged-in work while the ExtResource work was in transit.
- Legacy-Id: 18192
2020-07-20 01:58:26 +00:00
Jennifer Richards abe97eeb28 Make docs in Auth48 state visually distinct. Fixes #2925. Commit ready for merge.
- Legacy-Id: 18189
2020-07-17 18:14:23 +00:00
Henrik Levkowetz 4bc6a20661 Merged in [18162] from jennifer@painless-security.com:
Notify IRSG when an IRSG ballot is created. Fixes #2978.
 - Legacy-Id: 18177
Note: SVN reference [18162] has been migrated to Git commit 6e97a89786
2020-07-15 14:52:38 +00:00
Henrik Levkowetz 4fa431651c Renumbered some migrations to avoid multiple leaf nodes.
- Legacy-Id: 18169
2020-07-14 17:40:41 +00:00
Henrik Levkowetz 4b1074483a Merged in [18159] from jennifer@painless-security.com:
Notify secretariat when conflict review/status change doc enters an announcement pending state. Fixes #2962.
 - Legacy-Id: 18168
Note: SVN reference [18159] has been migrated to Git commit 24140fac50
2020-07-14 12:25:47 +00:00
Henrik Levkowetz 2d6925f02f Fixed an UnboundLocalError issue.
- Legacy-Id: 18167
2020-07-14 12:06:14 +00:00
Henrik Levkowetz a7b3d9c83b Merged in [18157] from jennifer@painless-security.com:
Store Auth48 URL as DocumentURL and display in RFC-Editor state. Migrates old data when possible. Alternative to 17563. Fixes #2722.
 - Legacy-Id: 18166
Note: SVN reference [18157] has been migrated to Git commit fff927b085
2020-07-14 11:22:11 +00:00
Robert Sparks 6227162e53 Updated mgmt command to find github repos to back up. Improved test coverage of branch changes. Branch ready for merge.
- Legacy-Id: 18163
2020-07-13 17:36:34 +00:00
Jennifer Richards 6e97a89786 Notify IRSG when an IRSG ballot is created. Fixes #2978. Commit ready for merge.
- Legacy-Id: 18162
2020-07-13 16:45:45 +00:00
Jennifer Richards 24140fac50 Notify secretariat when conflict review/status change doc enters an announcement pending state. Fixes #2962. Commit ready for merge.
- Legacy-Id: 18159
2020-07-13 16:11:58 +00:00
Robert Sparks 99df95d542 Cleaned up created names. Improved validation during migration. Cleaned up migration output. Cleaned the last of the awp includes from secr views. Removed now unused secr templates. Build extresource urls correctly during submission and when creating group wikis.
- Legacy-Id: 18158
2020-07-13 16:03:39 +00:00
Jennifer Richards fff927b085 Store Auth48 URL as DocumentURL and display in RFC-Editor state. Migrates old data when possible. Alternative to 17563. Fixes #2722. Commit ready for merge.
- Legacy-Id: 18157
2020-07-13 15:34:01 +00:00
Robert Sparks 53f7bc3ce6 migrated forward
- Legacy-Id: 18144
2020-07-09 21:07:05 +00:00
Robert Sparks 066ee27505 preparing to merge forward
- Legacy-Id: 18143
2020-07-09 20:29:45 +00:00
Robert Sparks c6f55a3306 Fixed some test scaffolding that used non-existent keys (uncovered by recent improvements in sqlite3 integrity checking). Commit ready for merge.
- Legacy-Id: 18130
2020-07-02 20:05:02 +00:00
Henrik Levkowetz 47a2174e80 Django 2.2 does not wrap single queries in transactions, for performance reasons. This caused some template tags that did database lookups to trigger exceptions. Fixed by moving the lookups (which would not normally change between apache reloads) out from the template tag code to module scope. Adding new groups of type ['ag','area','team','dir','program'] will now require a reload to show up in the group menu.
- Legacy-Id: 18086
2020-06-27 17:28:50 +00:00
Henrik Levkowetz dc5ca4eebc Merged in Django 2.1 upgrade changes.
- Legacy-Id: 18060
2020-06-24 20:51:00 +00:00
Henrik Levkowetz dc7581fef4 Changed NullBoleanField() to BooleanField(null=True), according to the 2.1 upgrade recommendations.
- Legacy-Id: 18051
2020-06-24 20:33:57 +00:00
Robert Sparks bf53bf1057 Merged forward. Added some missing parts to the XMPPURLValidator.
- Legacy-Id: 18050
2020-06-24 19:01:45 +00:00
Robert Sparks f7b1ce1fce Remove abandoned charters from any future telechats. Fixes #3003. Commit ready for merge.
- Legacy-Id: 18037
2020-06-22 21:22:54 +00:00
Robert Sparks d87a1d009b Map some new urls. Create a new external resource type
- Legacy-Id: 17873
2020-05-21 21:46:51 +00:00
Robert Sparks 8a0578040e Added better self-descriptive __str__ functions to PersonExtResource, GroupExtResource, and DocExtResource
- Legacy-Id: 17871
2020-05-21 20:59:44 +00:00
Robert Sparks 02e1226a03 Added appropriate Admin classes
- Legacy-Id: 17870
2020-05-21 20:48:03 +00:00
Robert Sparks 785f6243f1 explicit resource names for classes with the word resource in their name
- Legacy-Id: 17857
2020-05-20 18:28:05 +00:00
Robert Sparks 7e57be2bd3 merged forward ^/personal/rjs/explore-extref
- Legacy-Id: 17840
2020-05-19 18:47:47 +00:00
Henrik Levkowetz bc05e7a8d0 Refactored many instances of msg.get_payload(decode=True).decode('utf-8') and similar to get_payload_text(msg), which uses msg.get_charset() to get the actual charset to do the decoding, in a consistent manner. Also removed some instances of force_str, but more remain.
- Legacy-Id: 17807
2020-05-16 20:20:05 +00:00
Henrik Levkowetz 02804ba163 Fixed a case of too few format string parameters.
- Legacy-Id: 17768
2020-05-10 14:14:50 +00:00
Henrik Levkowetz 81197f044c Tweaked a 404 message to differentiate between two not-found cases.
- Legacy-Id: 17767
2020-05-10 14:13:39 +00:00
Henrik Levkowetz 29ccca849a Added a second link to htmlized draft content (on the datatracker) in addition to the link to the tools hosted htmlized document.
- Legacy-Id: 17760
2020-05-06 18:23:47 +00:00
Henrik Levkowetz 0b51e9a2b3 Removed a debug statement.
- Legacy-Id: 17732
2020-05-04 20:59:44 +00:00
Henrik Levkowetz 35a2a145fa Merged in [17708] from rjsparks@nostrum.com:
Remove 'point raised - writup needed' from the set of substates the IESG can use with a document. Fixes #2968.
 - Legacy-Id: 17731
Note: SVN reference [17708] has been migrated to Git commit fa492c3947
2020-05-04 20:50:17 +00:00
Henrik Levkowetz 3e6a1cfb11 Limited the data for 'New draft revisions over time' on the personal profile page to type='draft'.
- Legacy-Id: 17720
2020-05-01 11:37:32 +00:00
Robert Sparks fa492c3947 Remove "point raised - writup needed" from the set of substates the IESG can use with a document. Fixes #2968. Commit ready for merge.
- Legacy-Id: 17708
2020-04-29 20:32:31 +00:00
Robert Sparks 7587d564d4 Added GroupExtResources to the group about page, and added the ability to edit them.
- Legacy-Id: 17685
2020-04-24 21:47:24 +00:00
Robert Sparks 2de9eb93b5 Added validation to the migrations. Added external resources to the draft display page. Built an editor for DocExternalResources.
- Legacy-Id: 17683
2020-04-24 16:14:43 +00:00
Henrik Levkowetz 9aeda11a29 Removed some Py2 compatibility decorators.
- Legacy-Id: 17673
2020-04-21 13:54:31 +00:00
Robert Sparks 672f9bce26 Added display-name. Refined migrations.
- Legacy-Id: 17646
2020-04-15 19:54:29 +00:00
Henrik Levkowetz f10ddadc0e Merged in changes from trunk up to r17584.
- Legacy-Id: 17593
2020-04-07 16:02:52 +00:00
Henrik Levkowetz 98fab6c05b Merged in [17562] from jennifer@painless-security.com:
Add tooltips with doc name to 'updates' and 'obsoletes' links. Fixes #2866;
 - Legacy-Id: 17569
Note: SVN reference [17562] has been migrated to Git commit 2d6179c1a5
2020-03-31 13:03:15 +00:00
Jennifer Richards 2d6179c1a5 Add tooltips with doc name to "updates" and "obsoletes" links. Fixes #2866; Commit ready for merge.
- Legacy-Id: 17562
2020-03-30 20:22:05 +00:00
Henrik Levkowetz 37f39163d1 Prevent an exception on missing author.email.person when listing author emails.
- Legacy-Id: 17554
2020-03-27 16:14:29 +00:00
Henrik Levkowetz 50bb971fc3 Merged in [17496] from rjsparks@nostrum.com:
Remove the rest of the log.assertions checking that iesg_state existed in places we expected it to.  Removed unnecessary imports.
 - Legacy-Id: 17522
Note: SVN reference [17496] has been migrated to Git commit db5d11ea32
2020-03-23 22:34:03 +00:00
Henrik Levkowetz c6be826113 Merged in [17495] from rjsparks@nostrum.com:
Remove log.assertion that was checking that we covered the edges when we changed documents to always have an iesg state.
 - Legacy-Id: 17517
Note: SVN reference [17495] has been migrated to Git commit f2b106cf8c
2020-03-23 11:16:19 +00:00
Henrik Levkowetz 970248a3d6 Merged in [17494] from rjsparks@nostrum.com:
Use current email addresses when we have them when listing document authors. Fixes #1902.
 - Legacy-Id: 17516
Note: SVN reference [17494] has been migrated to Git commit 8583a0a098
2020-03-22 23:55:49 +00:00
Robert Sparks c055d37dcb Removed extref app. Refined population migrations.
- Legacy-Id: 17509
2020-03-22 19:27:28 +00:00
Robert Sparks db5d11ea32 Remove the rest of the log.assertions checking that iesg_state existed in places we expected it to. Commit ready for merge.
- Legacy-Id: 17496
2020-03-21 22:11:33 +00:00
Robert Sparks f2b106cf8c Remove log.assertion that was checking that we covered the edges when we changed documents to always have an iesg state. Commit ready for merge.
- Legacy-Id: 17495
2020-03-21 22:07:15 +00:00
Robert Sparks 8583a0a098 Use current email addresses when we have them when listing document authors. Fixes #1902. Commit ready for merge.
- Legacy-Id: 17494
2020-03-21 21:36:58 +00:00
Henrik Levkowetz 87bf4e8d58 Make sure to strip possible mail header field values of whitespace before applying email.utils.unquite(). Resolution by kivinen@iki.fi, Fixes issue #2899.
- Legacy-Id: 17485
2020-03-21 17:34:37 +00:00
Robert Sparks 316dd183d9 first run at external resources
- Legacy-Id: 17470
2020-03-19 23:32:24 +00:00
Henrik Levkowetz 08130f4986 Merged in [17396] from rjsparks@nostrum.com:
Move charters for replaced groups to a new replaced state. Close any outstanding ballots on them. Fixes #2889, #2873, and #1286.
 - Legacy-Id: 17420
Note: SVN reference [17396] has been migrated to Git commit 79bb245f8a
2020-03-11 19:21:43 +00:00
Henrik Levkowetz ec13af6c43 Added a guard against accessing attributes of None.
- Legacy-Id: 17409
2020-03-08 13:18:58 +00:00
Henrik Levkowetz 638fbc592e Added a data migration to fix up some incorrect external URLs for reviews.
- Legacy-Id: 17402
2020-03-07 14:21:41 +00:00
Robert Sparks 79bb245f8a Move charters for replaced groups to a new replaced state. Close any outstanding ballots on them. Fixes #2889, #2873, and #1286. Commit ready for merge.
- Legacy-Id: 17396
2020-03-06 16:47:59 +00:00
Henrik Levkowetz 726fcbf27d Removed all __future__ imports.
- Legacy-Id: 17391
2020-03-05 23:53:42 +00:00
Henrik Levkowetz ab173cb2eb Removed further six usage: six.string_types, six.integer_types, six.moves.urllib instances.
- Legacy-Id: 17390
2020-03-05 20:46:58 +00:00
Henrik Levkowetz e9a37d8ac8 Removed six.text_type(), changed six.moves.urllib to plain urllib, and removed now unused six imports.
- Legacy-Id: 17385
2020-03-05 14:41:41 +00:00
Henrik Levkowetz a14e38a1a1 Replaced most cases of using of urlopen(), instead using the higher-level 'requests' module where it simplifies the code.
- Legacy-Id: 17380
2020-03-03 17:34:42 +00:00
Henrik Levkowetz 94e4dc7c74 Python3 is more ticklish about comparing strings to None than Py2. Fixed an issue with this in generate_sort_key() for document searches.
- Legacy-Id: 17373
2020-03-02 12:47:05 +00:00
Henrik Levkowetz 07747b26c3 Fixed an issue with tests mocking urlopen() responses, and tweaked error messages on bad results in sync/rfceditor.py
- Legacy-Id: 17356
2020-02-27 15:05:27 +00:00
Henrik Levkowetz bea0c59d6e Fixed a string vs list issue for Message from form case.
- Legacy-Id: 17338
2020-02-26 17:01:00 +00:00
Henrik Levkowetz a7baacf519 Added missing .as_strings() to some gather_address_lists() calls, in order to assign strings rather than lists to Message CharField instances. Fixed a Message content_type setting.
- Legacy-Id: 17337
2020-02-26 16:59:18 +00:00
Henrik Levkowetz ab34ecbb58 Merged in [17288] from rjsparks@nostrum.com:
Make internal review email about an RG use the IRSG instead of the IESG in text and addresses. Fixes #2881.
 - Legacy-Id: 17307
Note: SVN reference [17288] has been migrated to Git commit 4cce1b5e08
2020-02-18 22:04:15 +00:00
Henrik Levkowetz 0a6260cf16 Merged in [17286] from rjsparks@nostrum.com:
Add Reply-To to the default group review text. Fixes #2883.
 - Legacy-Id: 17292
Note: SVN reference [17286] has been migrated to Git commit 7e09dea02c
2020-02-13 00:33:37 +00:00
Henrik Levkowetz 5e6ea5b2e8 Merged in [17284] from jennifer@painless-security.com:
Create DocEvent when related IprDisclosure is posted or removed. Fixes #2811.
 - Legacy-Id: 17291
Note: SVN reference [17284] has been migrated to Git commit b02b42da81
2020-02-12 23:11:26 +00:00
Robert Sparks 4cce1b5e08 Make internal review email about an RG use the IRSG instead of the IESG in text and addresses. Fixes #2881. Commit ready for merge.
- Legacy-Id: 17288
2020-02-11 20:17:42 +00:00
Robert Sparks 7e09dea02c Add Reply-To to the default group review text. Fixes #2883. Commit ready for merge.
- Legacy-Id: 17286
2020-02-11 15:39:20 +00:00
Jennifer Richards b02b42da81 Create DocEvent when related IprDisclosure is posted or removed. Fixes #2811. Commit ready for merge.
- Legacy-Id: 17284
2020-02-07 20:41:13 +00:00
Henrik Levkowetz 32e9692f54 Fixed a py2/py3 issue in sync.rfceditor.post_approved_draft(), and enhanced tests to execute the failing code (with Mock functions) instead of skipping the code.
- Legacy-Id: 17265
2020-01-28 22:37:50 +00:00
Henrik Levkowetz 82baf773d8 Merged in [17228] from rjsparks@nostrum.com:
Improve admin handling of DocumentURL. Fixes #2865.
 - Legacy-Id: 17231
Note: SVN reference [17228] has been migrated to Git commit 0040a85395
2020-01-14 13:43:56 +00:00
Henrik Levkowetz 3fe48041da Merged in [17227] from rjsparks@nostrum.com:
Allow document Additional URLs to become empty. Fixes #2864.
 - Legacy-Id: 17229
Note: SVN reference [17227] has been migrated to Git commit 0c0cc5dcf5
2020-01-14 11:09:21 +00:00
Robert Sparks 0040a85395 Improve admin handling of DocumentURL. Fixes #2865. Commit ready for merge.
- Legacy-Id: 17228
2020-01-13 19:33:35 +00:00
Robert Sparks 0c0cc5dcf5 Allow document Additional URLs to become empty. Fixes #2864. Commit ready for merge.
- Legacy-Id: 17227
2020-01-13 19:03:58 +00:00
Henrik Levkowetz 288f4135e3 Added (currently inactive) code to trigger exceptions on failure to resolve template variables during tests, and added fixes for a couple of places that triggered such exceptions.
- Legacy-Id: 17226
2020-01-13 17:16:43 +00:00
Henrik Levkowetz d2b2bb1417 Merged in [17219] from rjsparks@nostrum.com:
Update some uses of pos.ad in templates to pos.balloter. Fixes #2863.
 - Legacy-Id: 17224
Note: SVN reference [17219] has been migrated to Git commit af0a9954fe
2020-01-13 15:01:40 +00:00
Robert Sparks af0a9954fe Update some uses of pos.ad in templates to pos.balloter. Fixes #2863. Commit ready for merge.
- Legacy-Id: 17219
2020-01-10 19:04:42 +00:00
Henrik Levkowetz ba3010e3b0 Added fetch and use of the RFC-Editor's errata.json file in order to set a new 'verified-errata' document tag, and use that to show the link to inline-errata documents only when there is verified errata. Fixes issue #2861.
- Legacy-Id: 17212
2020-01-10 12:22:33 +00:00
Henrik Levkowetz acaca62195 Merged in [17179] from housley@vigilsec.com:
Add references and referened-by buttons to each RFC listed on the peron profile page.  In addition, changed the buttons on the document references and document referenced-by pages to match the buttons on the main document page (the buttons with the long arrows). Fixes #2854, although it does not include the number of references in the button.
 - Legacy-Id: 17191
Note: SVN reference [17179] has been migrated to Git commit 3460f9242f
2020-01-07 15:29:40 +00:00
Henrik Levkowetz b14cba5f15 Refactored some document methods to align better with Django's get_absolute_url(), and added Person.get_absolute_uri()
- Legacy-Id: 17189
2020-01-07 13:33:02 +00:00
Henrik Levkowetz b2fc68a978 Merged in [17177] from rjsparks@nostrum.com:
Improve can_ballot. Fixes #2856.
 - Legacy-Id: 17188
Note: SVN reference [17177] has been migrated to Git commit eb69b9e14b
2020-01-07 11:47:03 +00:00
Henrik Levkowetz 2961f4dfba Merged in [17173] from rjsparks@nostrum.com:
Works around a problem with displaying IESG evaluation information for non-approve ballot types. Partially addresses #2851.
 - Legacy-Id: 17183
Note: SVN reference [17173] has been migrated to Git commit 5773d62c1b
2020-01-06 18:48:33 +00:00
Robert Sparks eb69b9e14b Improve can_ballot. Fixes #2856. Commit ready for merge.
- Legacy-Id: 17177
2020-01-03 15:55:48 +00:00
Robert Sparks 5773d62c1b Works around a problem with displaying IESG evaluation information for non-approve ballot types. Partially addresses #2851. Commit ready for merge.
- Legacy-Id: 17173
2019-12-23 19:48:21 +00:00
Henrik Levkowetz 6482254a03 Merged in ^/branch/akayla/irsg/6.113.1.dev0 from peter@akayla.com.
This provides support for IRSG ballots, similar to the IESG ballots support which has been in use for quite some time.  The IRSG ballots differ from IESG ballots in a number of ways, described in detail in the RFP and SoW for this work and implemented here.
 - Legacy-Id: 17164
2019-12-20 15:00:55 +00:00
Henrik Levkowetz 1d0e8b7527 Merged in [17148] from rjsparks@nostrum.com:
Only gather addresses instead of name_addrs for the group_secretaries Mailtrigger Recipient. Fixes #2847.
 - Legacy-Id: 17156
Note: SVN reference [17148] has been migrated to Git commit d22da6259b
2019-12-18 18:29:26 +00:00
Henrik Levkowetz 1e1bd2f995 Merged in [17147] from rjsparks@nostrum.com:
Show diff controls for a bis -00 document. Fixes #2846.
 - Legacy-Id: 17155
Note: SVN reference [17147] has been migrated to Git commit 18b8a795ad
2019-12-18 16:33:59 +00:00
Henrik Levkowetz 06fe583351 Merged /branch/iola/meeting-improvement-r16992@17152 from olau@iola.dk:
* Renamed Meeting.agenda to Meeting.schedule together with a bunch of related
   internal things

* Moved Session.status, .requested, and .requested_by to a new SchedulingEvent.

* Turned sessions into regular sessions and non-sessions into misc.
  sessions in the UI and code to avoid ambiguity.

  This doesn't change the data in the DB except for uses of
  TimeSlotTypeName where 'session' is now 'regular'.
 - Legacy-Id: 17153
2019-12-17 23:43:07 +00:00
Peter E. Yee 8313b0adbf Fixes for questions for you. Does not include any changes in docker/
- Legacy-Id: 17150
2019-12-13 23:01:06 +00:00
Peter E. Yee bb7e504d14 12/13 merged into 6.113.1.dev0 with migration ordering failure
- Legacy-Id: 17149
2019-12-13 19:53:45 +00:00
Robert Sparks d22da6259b Only gather addresses instead of name_addrs for the group_secretaries Mailtrigger Recipient. Fixes #2847. Commit ready for merge.
- Legacy-Id: 17148
2019-12-13 18:37:30 +00:00
Robert Sparks 18b8a795ad Show diff controls for a bis -00 document. Fixes #2846. Commit ready for merge.
- Legacy-Id: 17147
2019-12-12 17:17:51 +00:00
Ole Laursen de2b6d4072 Fix use of status still referring to the old attribute in the group
meeting page
 - Legacy-Id: 17142
2019-12-09 18:08:37 +00:00
Ole Laursen 568670c060 Turn sessions into regular sessions and non-sessions into misc.
sessions in the UI and code to avoid ambiguity.

This doesn't change the data in the DB except for uses of
TimeSlotTypeName where 'session' is now 'regular'.
 - Legacy-Id: 17128
2019-12-06 20:02:26 +00:00
Ole Laursen 3957743b85 Move Session.status, .requested, and .requested_by to a new SchedulingEvent
- Legacy-Id: 17122
2019-12-05 12:41:09 +00:00
Henrik Levkowetz fcb6806d17 Merged in work from sasha@dashcare.nl on Review Queue Managemnt:
This abstracts queue management, making it possible to implement different
policies for each team. It provides two concrete policies:
RotateAlphabeticallyReviewerQueuePolicy, which rotates an alphabetically
ordered reviewer list with consideration for skip indications, and is the
default policy; and LeastRecentlyUsedReviewerQueuePolicy, a simple
least-recently-used policy.  Also see issues #2721 and #2656.
 - Legacy-Id: 17121
2019-12-04 23:02:52 +00:00
Henrik Levkowetz fc0060079a Merged in [17085] from sasha@dashcare.nl:
Fix #2278 - Notify secretary when a review request is closed.
The assigned reviewer was already notified, but secretaries were always
excluded.
 - Legacy-Id: 17108
Note: SVN reference [17085] has been migrated to Git commit 56003f66d8
2019-11-27 22:19:58 +00:00
Henrik Levkowetz 2c28cb3eb0 When group secretary email addresses occurred in some mailtrigger To: entries, they were given with both name and address parts, while when occurring in Cc: entries, no names were present. Fixed one case of this, and adjusted a test case for consistency. Also added a more general utility function to exclude selected email addresses from email address lists.
- Legacy-Id: 17107
2019-11-27 21:49:22 +00:00
Henrik Levkowetz 5cf2cac8d0 Added support for bibxml3 entries using the traditional reference entry name (with 'reference.I-D....'), instead of just the plain draft name.
- Legacy-Id: 17106
2019-11-27 15:31:40 +00:00
Henrik Levkowetz 2bc5606986 Merged in [17070] from sasha@dashcare.nl:
Fix #2061 - Improve 'complete review' workflow for secretaries.
When a secretary completes a review, 'link to a review message'
is automatically selected, and the first non-reply mail is used to
fill in the review details. The secretary can still modify all details.
The order of fields for secretaries is also modified to fit this workflow.
All cases where 'link to review message' is used, by reviewers or
secretaries, now attempt to fill in the 'reviewed version' if found
in the email subject.
 - Legacy-Id: 17105
Note: SVN reference [17070] has been migrated to Git commit dc9546211f
2019-11-26 20:22:56 +00:00
Henrik Levkowetz 7ec09554bb Merged in [17069] from sasha@dashcare.nl:
Fix #2074, fix #2358 - Add history for assignments, requests and unavailability.
 - Legacy-Id: 17104
Note: SVN reference [17069] has been migrated to Git commit 65d84155b6
2019-11-26 16:09:43 +00:00
Henrik Levkowetz 47c3afc328 Added a test for bibtex page content-type.
- Legacy-Id: 17103
2019-11-26 01:34:35 +00:00
Henrik Levkowetz 8364e119bc Fixed an incorrect content_type
- Legacy-Id: 17102
2019-11-25 18:47:58 +00:00
Sasha Romijn 10a7bee637 Merge reviewer-queue-management on top of 6.111.1.dev0
- Legacy-Id: 17094
2019-11-22 12:35:28 +00:00
Robert Sparks a33ba86452 Add pages to assignment email. Fixes #2838. Commit ready for merge.
- Legacy-Id: 17090
2019-11-21 22:41:11 +00:00
Sasha Romijn 56003f66d8 Fix #2278 - Notify secretary when a review request is closed.
The assigned reviewer was already notified, but secretaries were always
excluded.

Commit ready for merge.
 - Legacy-Id: 17085
2019-11-21 11:03:21 +00:00
Henrik Levkowetz 424dcfb42d Some tweaks related to the changes related to #2830.
- Legacy-Id: 17076
2019-11-21 06:01:56 +00:00
Henrik Levkowetz f0adebbe70 Merged in [17062] from rjsparks@nostrum.com:
Change the default template for assigned review emails to name the assigned reviewer. Fixes #2830.
 - Legacy-Id: 17073
Note: SVN reference [17062] has been migrated to Git commit 38e73b6f89
2019-11-21 01:32:10 +00:00
Sasha Romijn dc9546211f Fix #2061 - Improve "complete review" workflow for secretaries.
When a secretary completes a review, "link to a review message"
is automatically selected, and the first non-reply mail is used to
fill in the review details. The secretary can still modify all details.
The order of fields for secretaries is also modified to fit this workflow.

All cases where "link to review message" is used, by reviewers or
secretaries, now attempt to fill in the "reviewed version" if found
in the email subject.

Commit ready for merge.
 - Legacy-Id: 17070
2019-11-20 14:25:05 +00:00
Sasha Romijn 65d84155b6 Fix #2074, fix #2358 - Add history for assignments, requests and unavailability.
Commit ready for merge.
 - Legacy-Id: 17069
2019-11-20 11:29:14 +00:00
Henrik Levkowetz 19eb5de983 Merged in [17044] from valery@smyslov.net:
Make it clearer that a chair can't revise materials for a meeting once the proceedings deadline has passed

    Fixes issue #2612.
 - Legacy-Id: 17067
Note: SVN reference [17044] has been migrated to Git commit 182dae28a8
2019-11-20 10:10:31 +00:00
Robert Sparks 38e73b6f89 Change the default template for assigned review emails to name the assigned reviewer. Fixes #2830. Commit ready for merge.
- Legacy-Id: 17062
2019-11-19 21:27:23 +00:00
Sasha Romijn 0c0980c641 Fix #2420 - Add reviewer back to top of the queue after rejected/withdrawn reviews.
- Legacy-Id: 17058
2019-11-19 10:57:56 +00:00
Sasha Romijn 8bb6955d47 Fix #2418 - Account for previous rejected reviews in recommended assignment order
- Legacy-Id: 17053
2019-11-18 19:50:33 +00:00
Valery Smyslov 182dae28a8 Fixes issue #2612. Commit ready for merge.
- Legacy-Id: 17044
2019-11-18 08:48:15 +00:00
Henrik Levkowetz 69069d12c6 Merged in [17027] from rcross@amsl.com:
Fix issue where draft resurrect feature does not move most recent draft file from the archive directory back to the current draft directory. Fixes #2646.
 - Legacy-Id: 17032
Note: SVN reference [17027] has been migrated to Git commit 9882d7fda8
2019-11-16 07:58:37 +00:00
Ryan Cross 9882d7fda8 Fix issue where draft resurrect feature does not move most recent draft file from the archive directory back to the current draft directory. Fixes #2646. Commit ready for merge
- Legacy-Id: 17027
2019-11-16 05:15:03 +00:00
Sasha Romijn c36fcdc5a7 Update update_policy_state_for_assignment for new policies, fix tests,
fix some other minor things.
 - Legacy-Id: 17023
2019-11-13 14:39:42 +00:00
Henrik Levkowetz 68b0e42e94 Tweaked a test person name to match the usage in the code tested.
- Legacy-Id: 17020
2019-11-12 18:22:42 +00:00
Henrik Levkowetz 4049804920 Merged in [16981] from rjsparks@nostrum.com:
Look at the tree of replaces-relationships when determining if some reviewer has reviewed this (or perhaps an ancestor) of this document. Fixes #2821.
 - Legacy-Id: 16992
Note: SVN reference [16981] has been migrated to Git commit f740adcfc1
2019-11-11 16:57:40 +00:00
Sasha Romijn 1c84e3c363 Add additional tests for queue policies and fix unused import.
- Legacy-Id: 16990
2019-11-11 16:32:49 +00:00
Sasha Romijn ce812a3a4f - Make skipping unavailable reviews the default, except for the
reviewer_overview page.
- Make default_reviewer_rotation_list use a consistent return type
 - Legacy-Id: 16986
2019-11-11 14:47:37 +00:00
Henrik Levkowetz 20eb9d8ac1 Merged in [16939] from sasha@dashcare.nl:
Fix #2050 - Allow adding review wishes from document and search pages.
On the main page of a document and in document search results, a new
button allows review team members to add a review wish for that document.

For reviewers that are only on one team, this essentially works
identical to tracking a document. Reviewers that are on multiple teams
are lead through an intermediate step to select a review team, and then
returned to their search or document page.
 - Legacy-Id: 16985
Note: SVN reference [16939] has been migrated to Git commit 6e55f26dbd
2019-11-11 14:22:54 +00:00
Sasha Romijn b5a31c3c6a Update some terminology and docstrings.
- Legacy-Id: 16983
2019-11-11 12:31:16 +00:00
Robert Sparks f740adcfc1 Look at the tree of replaces-relationships when determining if some reviewer has reviewed this (or perhaps an ancestor) of this document. Fixes #2821. Commit ready for merge.
- Legacy-Id: 16981
2019-11-07 23:02:19 +00:00
Henrik Levkowetz b7724c43d7 Changed some # type: ignore statements to work with the latest mypy
- Legacy-Id: 16968
2019-11-05 22:41:18 +00:00
Henrik Levkowetz c565dca35b Fixed some .format() calls.
- Legacy-Id: 16967
2019-11-05 22:38:54 +00:00
Henrik Levkowetz 5d8aa656a5 Added code to catch bad rev numbers for bibxml3 requests.
- Legacy-Id: 16963
2019-11-05 18:24:45 +00:00
Henrik Levkowetz c233f07b5d Added a management command to generate draft bibxml files, and also a trial version of datatracker draft bibxml pages.
- Legacy-Id: 16962
2019-11-05 18:10:29 +00:00
Sasha Romijn e518824a69 Initial refactoring of the current reviewer assignment code.
- Legacy-Id: 16961
2019-11-05 16:39:31 +00:00
Sasha Romijn eab14ea1c5 Early work to extract reviewer policy from review/utils.py.
- Legacy-Id: 16950
2019-10-31 15:01:14 +00:00
Henrik Levkowetz 0f12d477ad Merged in [16927] from sasha@dashcare.nl:
Refs #2217 - Small cleanup from changeset [16924]
 - Legacy-Id: 16947
Note: SVN reference [16924] has been migrated to Git commit 871a4b653b

Note: SVN reference [16927] has been migrated to Git commit 730e64d253
2019-10-30 17:02:01 +00:00
Sasha Romijn 6e55f26dbd Fix #2050 - Allow adding review wishes from document and search pages.
On the main page of a document and in document search results, a new
button allows review team members to add a review wish for that document.

For reviewers that are only on one team, this essentially works
identical to tracking a document. Reviewers that are on multiple teams
are lead through an intermediate step to select a review team, and then
returned to their search or document page.

Commit ready for merge.
 - Legacy-Id: 16939
2019-10-29 16:27:56 +00:00
Henrik Levkowetz 63484a8cb5 Merged in [16937] from rjsparks@nostrum.com:
Only show a blocked document once in the set of blocked documents on an ADs document page. Fixes #2818.
 - Legacy-Id: 16938
Note: SVN reference [16937] has been migrated to Git commit 8f1e9990cf
2019-10-29 15:03:01 +00:00
Henrik Levkowetz c606461b22 Merged in [16924] from sasha@dashcare.nl:
Fix #2217 - Allow submission of unsolicited reviews by secretaries.
- For team secretaries, a button 'Submit unsolicited review' will now
  appear next to 'Request review' on the document's main page.
- If the secretary is a secretary for multiple teams, they are taken
  through an intermediate page to select for which team they are
  submitting their review.
- The form is similar (and using the same code) as the usual review
  completion, with a few extra fields for the review type and reviewer,
  which would usually already be known.
- When submitting the review, a ReviewRequest and ReviewAssignment are
  automatically created. The assignment is then immediately closed in
  the usual way.
- Other workflows are unchanged.
The issues with the review form in #2061 are slightly worse for the
unsolicited review scenario, but that will be improved when #2061 is
fixed.
 - Legacy-Id: 16932
Note: SVN reference [16924] has been migrated to Git commit 871a4b653b
2019-10-26 21:16:53 +00:00
Henrik Levkowetz 00fb8d4b43 Merged in [16923] from sasha@dashcare.nl:
Fix #2087 - Show doc authors and WG chairs in unassigned reviews views.
 - Legacy-Id: 16931
Note: SVN reference [16923] has been migrated to Git commit 93797b3fc1
2019-10-26 20:26:21 +00:00
Henrik Levkowetz 09c34fa131 Merged in [16921] from sasha@dashcare.nl:
Fix #2123 - Use a DBTemplate when a review assignment is mailed.
The message sent to a reviewer when a new review is assigned to them,
is now taken from a DBTemplate, allowing it to be customised
per team. This includes a migration to create a default template,
which looks the same as before.
 - Legacy-Id: 16928
Note: SVN reference [16921] has been migrated to Git commit 9df5839874
2019-10-26 15:16:18 +00:00
Sasha Romijn 730e64d253 Refs #2217 - Small cleanup from changeset [16924]
Commit ready for merge.
 - Legacy-Id: 16927
Note: SVN reference [16924] has been migrated to Git commit 871a4b653b
2019-10-26 08:04:31 +00:00
Sasha Romijn 871a4b653b Fix #2217 - Allow submission of unsolicited reviews by secretaries.
- For team secretaries, a button "Submit unsolicited review" will now
  appear next to "Request review" on the document's main page.
- If the secretary is a secretary for multiple teams, they are taken
  through an intermediate page to select for which team they are
  submitting their review.
- The form is similar (and using the same code) as the usual review
  completion, with a few extra fields for the review type and reviewer,
  which would usually already be known.
- When submitting the review, a ReviewRequest and ReviewAssignment are
  automatically created. The assignment is then immediately closed in
  the usual way.
- Other workflows are unchanged.

The issues with the review form in #2061 are slightly worse for the
unsolicited review scenario, but that will be improved when #2061 is
fixed.

Commit ready for merge.
 - Legacy-Id: 16924
2019-10-24 12:37:59 +00:00
Sasha Romijn 93797b3fc1 Fix #2087 - Show doc authors and WG chairs in unassigned reviews views.
Commit ready for merge.
 - Legacy-Id: 16923
2019-10-24 07:25:47 +00:00
Sasha Romijn 9df5839874 Fix #2123 - Use a DBTemplate when a review assignment is mailed.
The message sent to a reviewer when a new review is assigned to them,
is now taken from a DBTemplate, allowing it to be customised
per team. This includes a migration to create a default template,
which looks the same as before.

Commit ready for merge.
 - Legacy-Id: 16921
2019-10-23 16:06:44 +00:00
Henrik Levkowetz 3e9bad1c8b Merged in [16883] from sasha@dashcare.nl:
Fix #2277 - Do not allow reviewers to reject overdue reviews.
If a review request is past the deadline, reviewers will no longer be
able to reject the assignment.
 - Legacy-Id: 16909
Note: SVN reference [16883] has been migrated to Git commit 3c2b01b3ff
2019-10-22 20:59:13 +00:00
Henrik Levkowetz 6688d37bb4 Merged in [16875] from rjsparks@nostrum.com:
Adds docs with blocking positions to the ADs my-documents page. Fixes #2800.
 - Legacy-Id: 16906
Note: SVN reference [16875] has been migrated to Git commit f7d4600371b3f2a3c92aa184b517ed4ebee3a118
2019-10-22 17:43:54 +00:00
Henrik Levkowetz 04e342b3ce Merged in [16860] from rjsparks@nostrum.com:
Be resilient to statistics queries that provide a state but not a substate. Fixes #2813.
 - Legacy-Id: 16905
Note: SVN reference [16860] has been migrated to Git commit 654ae848f657957d2f7d259293ba523619d81cb5
2019-10-22 16:05:42 +00:00
Henrik Levkowetz fc2abcce20 Fixed a couple of places where document alias.document was used instead of alias.docs.all().
- Legacy-Id: 16901
2019-10-22 14:19:22 +00:00
Henrik Levkowetz 9e21c011f6 Merged in [16854] from rjsparks@nostrum.com:
Remove some pre-ReviewAssignment refactor logic, and simplify what remains while still allowing a group secretary to not assign a row on the unassigned requests form after touching the control to assign a reviewer. Fixes #2812.
 - Legacy-Id: 16900
Note: SVN reference [16854] has been migrated to Git commit b158807134e2d26b3e013b6414b6fcbaa0e39f77
2019-10-22 12:43:56 +00:00
Henrik Levkowetz 0313a4f58d Merged in [16836] from rjsparks@nostrum.com:
Changes to displaying the ballot_icon to account for IRSG ballots.
 - Legacy-Id: 16896
Note: SVN reference [16836] has been migrated to Git commit 6375aa3bedbd612cd519bf3dccced2465b145593
2019-10-21 19:37:36 +00:00
Sasha Romijn 3c2b01b3ff Fix #2277 - Do not allow reviewers to reject overdue reviews.
If a review request is past the deadline, reviewers will no longer be
able to reject the assignment.

Commit ready for merge.
 - Legacy-Id: 16883
2019-10-21 14:15:08 +00:00
Henrik Levkowetz 77abc3868e Type checking tweaks.
- Legacy-Id: 16867
2019-10-15 16:54:33 +00:00
Henrik Levkowetz 5460a5ab84 Merged in [16785] from rjsparks@nostrum.com:
Improve alerts when looking at non-current versions of a charter document. Fixes #2774.
 - Legacy-Id: 16845
Note: SVN reference [16785] has been migrated to Git commit 07637c27e1a76d3cbb92cc22b987b9f1426d8d52
2019-10-08 14:24:01 +00:00
Henrik Levkowetz a895e19882 Merged in [16770] from rjsparks@nostrum.com:
Populate completed_on when rejecting a ReviewAssignment. Migrate past rejected assignments accordingly. Fixes #2782.
 - Legacy-Id: 16841
Note: SVN reference [16770] has been migrated to Git commit 661e4c5369dc0d165bfcb56e3278906fe1de3b55
2019-10-08 12:46:49 +00:00
Henrik Levkowetz 1edc863969 Merged in [16763] from rjsparks@nostrum.com:
Allow responsible AD to be None for documents in iesg states idexists and dead. Fixes #2803 and #993.
 - Legacy-Id: 16838
Note: SVN reference [16763] has been migrated to Git commit 9f54af355d4e2b17404caa1755be74d22b5edad1
2019-10-07 19:54:56 +00:00
Henrik Levkowetz 8c71252fee Merged in [16688] from sasha@dashcare.nl:
Fix #2478 - When completing review, show which addresses it is posted to.
 - Legacy-Id: 16795
Note: SVN reference [16688] has been migrated to Git commit 68ea11a916
2019-10-01 15:02:03 +00:00
Henrik Levkowetz 25ac3b67f5 Merged in [16680] from sasha@dashcare.nl:
Fix #2354 - Make review_completed configurable per team and review type
This includes a migration to change mailtrigger slugs to be up to 64
characters instead of 32, as some slugs would not fit and require clunky
abbreviations.
A data migration creates triggers for existing teams, and they are also
created on the fly if a trigger does not exist yet, providing a safe
fallback for new review teams. The review_completed mailtrigger serves
as the template for new triggers.
This commit also includes tests for gather_address_lists(), as none
existed.
 - Legacy-Id: 16794
Note: SVN reference [16680] has been migrated to Git commit 1390ae073c
2019-10-01 13:35:52 +00:00
Henrik Levkowetz 70380cb8ea Merged in [16672] from sasha@dashcare.nl:
Fix #2328 - Use mailtriggers to find destinations in review app
As the review app has several conditionals that don't fit
entirely well within mailtriggers, the templates use a bit
of extra context to figure out who exactly to include. This also
extends the tests for review, to check for correct recipients.
It also adds a tiny feature to mailtrigger to entirely exclude
certain addresses, as required by the review-generated mails.
 - Legacy-Id: 16793
Note: SVN reference [16672] has been migrated to Git commit 88b7b45b0e
2019-10-01 13:06:48 +00:00
Henrik Levkowetz 632423aa6c Merged in [16670] from sasha@dashcare.nl:
Fix #2590 - Allow secretary to adjust date when completing a review.
This also fixes other issues identified in #2590, around the
modification of historical document events. The behaviour is now:
- When the assigned reviewer posts a review, a single event is
  created, set to current date/time.
- When the secretary records a review in the datatracker, they may
  set a different completion date, which is autofilled if an email
  is selected. One event is generated for the original completion
  date, and one for the secretary's action.
- Each revision generates a new event, rather than updating previous
  existing events.
 - Legacy-Id: 16791
Note: SVN reference [16670] has been migrated to Git commit 3942f9acc7
2019-10-01 12:05:04 +00:00
Henrik Levkowetz 10c8596664 Merged in [16667] from sasha@dashcare.nl:
Fix #2071 - Add optional comment field for close request.
 - Legacy-Id: 16786
Note: SVN reference [16667] has been migrated to Git commit 4669b0934b
2019-09-30 20:55:16 +00:00
Henrik Levkowetz 33e8733b91 Fixed up mypy issues or added type:ignore comments as needed for a clean mypy run.
- Legacy-Id: 16772
2019-09-30 15:42:18 +00:00
Henrik Levkowetz a3f59ee9a2 Added conversion of version 3 XML draft submissions to v3 HTML, in addition to text. Added a link to HTML versions (when present) to the document status page, and renamed the link to the htmlized version to 'htmlized'. Converted the submission XML test-file to version 3. Added test code to check that HTML is generated for version 3 XML submissions.
- Legacy-Id: 16737
2019-09-20 14:24:44 +00:00
Henrik Levkowetz 44f42cdb3a Undid the previous commit
- Legacy-Id: 16735
2019-09-20 11:30:04 +00:00
Henrik Levkowetz e3e9f676b7 Added class 'bg-danger' to submission error messages. The big 'Meta-Data errors found' pane on red background is so noticeable that class 'text-danger' which gives bold text is insufficient to stand out clearly. Adding 'bg-danger' gives the specific error messages red background, too.
- Legacy-Id: 16734
2019-09-20 11:28:53 +00:00
Henrik Levkowetz 4283740bf1 Added a link to RFCs rendered with inline errata (where available) to the /doc/rfcNNNN pages.
- Legacy-Id: 16732
2019-09-13 22:59:38 +00:00
Henrik Levkowetz 96f6f47758 Updated rfc-editor sync code to recognize the new Tooling Issue queue state.
- Legacy-Id: 16730
2019-09-11 16:20:25 +00:00
Henrik Levkowetz 3160f55597 Added code to catch a form error regularly caused by form spam and return a failure instead of triggering a server 500.
- Legacy-Id: 16728
2019-09-10 16:10:11 +00:00
Henrik Levkowetz 3336f4a11b Merged in [16656] from rjsparks@nostrum.com:
Add IANA expert review tracking.
 - Legacy-Id: 16716
Note: SVN reference [16656] has been migrated to Git commit e45a32e49f
2019-09-08 14:34:29 +00:00
Henrik Levkowetz 1c38387279 Merged in [16649] from rjsparks@nostrum.com:
Guard against attempts to use the review request view with a ReviewRequest id that does not match the document for the ReviewRequest. This would have exposed #2776 much earlier.
 - Legacy-Id: 16710
Note: SVN reference [16649] has been migrated to Git commit 01ceeba131
2019-09-05 22:59:59 +00:00
Henrik Levkowetz e49ff19255 Merged in [16615] from peter@akayla.com:
Spelling correction in a comment.  No open issue.
 - Legacy-Id: 16697
Note: SVN reference [16615] has been migrated to Git commit 7f6edfcabc
2019-09-03 12:52:26 +00:00
Sasha Romijn 68ea11a916 Fix #2478 - When completing review, show which addresses it is posted to.
Commit ready for merge.
 - Legacy-Id: 16688
2019-09-02 15:52:26 +00:00
Sasha Romijn 1390ae073c Fix #2354 - Make review_completed configurable per team and review type
This includes a migration to change mailtrigger slugs to be up to 64
characters instead of 32, as some slugs would not fit and require clunky
abbreviations.
A data migration creates triggers for existing teams, and they are also
created on the fly if a trigger does not exist yet, providing a safe
fallback for new review teams. The review_completed mailtrigger serves
as the template for new triggers.
This commit also includes tests for gather_address_lists(), as none
existed.

Commit ready for merge.
 - Legacy-Id: 16680
2019-08-30 17:40:55 +00:00
Henrik Levkowetz 7ad34ba69e The logic for displaying 'Email requested to be sent for earlier discuss' on IESG positions was wrong; it would count later discusses and discusses from other ADs as an 'earlier discuss'. Moved any_email_sent() from ballot to position, and refined the criteria.
- Legacy-Id: 16675
2019-08-28 20:56:34 +00:00
Sasha Romijn 88b7b45b0e Fix #2328 - Use mailtriggers to find destinations in review app
As the review app has several conditionals that don't fit
entirely well within mailtriggers, the templates use a bit
of extra context to figure out who exactly to include. This also
extends the tests for review, to check for correct recipients.
It also adds a tiny feature to mailtrigger to entirely exclude
certain addresses, as required by the review-generated mails.

Commit ready for merge.
 - Legacy-Id: 16672
2019-08-27 15:15:13 +00:00
Sasha Romijn 3942f9acc7 Fix #2590 - Allow secretary to adjust date when completing a review.
This also fixes other issues identified in #2590, around the
modification of historical document events. The behaviour is now:
- When the assigned reviewer posts a review, a single event is
  created, set to current date/time.
- When the secretary records a review in the datatracker, they may
  set a different completion date, which is autofilled if an email
  is selected. One event is generated for the original completion
  date, and one for the secretary's action.
- Each revision generates a new event, rather than updating previous
  existing events.
 - Legacy-Id: 16670
2019-08-26 12:58:31 +00:00
Sasha Romijn 4669b0934b Fix #2071 - Add optional comment field for close request. Commit ready for merge.
- Legacy-Id: 16667
2019-08-23 11:37:41 +00:00
Henrik Levkowetz 9a9c9fb426 Merged in [16603] from rjsparks@nostrum.com:
Remove a debug statement from the command
 - Legacy-Id: 16661
Note: SVN reference [16603] has been migrated to Git commit d27096f39022e9bee1ffa84631d5ae847af9be3e
2019-08-20 10:35:06 +00:00
Henrik Levkowetz 509e0da7ab Merged in [16601] from rjsparks@nostrum.com:
A management command to correct the state of several slide presentations uploaded during IETF105, and perform some missed powerpoint to pdf conversions.
 - Legacy-Id: 16659
Note: SVN reference [16601] has been migrated to Git commit a14188d95cf200dfbdd93485b89808f3336b0649
2019-08-19 17:50:56 +00:00
Robert Sparks e45a32e49f Add IANA expert review tracking. Commit ready for merge.
- Legacy-Id: 16656
2019-08-15 20:36:07 +00:00
Robert Sparks 01ceeba131 Guard against attempts to use the review request view with a ReviewRequest id that does not match the document for the ReviewRequest. This would have exposed #2776 much earlier. Commit ready for merge.
- Legacy-Id: 16649
2019-08-14 16:06:13 +00:00
Henrik Levkowetz 8e313e4ed5 Adjusted migrations for Python2/3 compatibility.
- Legacy-Id: 16633
2019-08-08 17:11:14 +00:00
Peter E. Yee 7f6edfcabc Spelling correction in a comment. No open issue. Commit ready for merge.
- Legacy-Id: 16615
2019-08-04 23:58:43 +00:00
Henrik Levkowetz 41c6bbf62e Merged in [16563] from housley@vigilsec.com:
Improve performance for a few pages
 - Legacy-Id: 16613
Note: SVN reference [16563] has been migrated to Git commit 765ce0d0b0
2019-08-04 15:44:06 +00:00
Henrik Levkowetz 0a591e4b2e Merged in [16562] from rcross@amsl.com:
Added encoding='utf-8' parameter to io.open operations in tests that starting failing since adding unicode_literals.
 - Legacy-Id: 16588
Note: SVN reference [16562] has been migrated to Git commit d332bfd91d
2019-07-24 23:44:36 +00:00
Henrik Levkowetz 7039520557 Merged in [16557] from pusateri@bangj.com:
Partially addresses issue #1926 still displaying markdown as text. Fixes #2704.
 - Legacy-Id: 16576
Note: SVN reference [16557] has been migrated to Git commit f76c46bef0
2019-07-22 18:09:53 +00:00
Russ Housley 765ce0d0b0 Improve performance for a few pages
- Legacy-Id: 16563
2019-07-20 22:59:26 +00:00