fix: appropriate metadata for editorial stream docs (#7195)
This commit is contained in:
Robert Sparks
GitHub
parent
b21706ef5c
commit
3bfb9303bc
|
|
@ -20,7 +20,7 @@ from django.utils.html import escape
|
||||||
import debug # pyflakes:ignore
|
import debug # pyflakes:ignore
|
||||||
|
|
||||||
from ietf.doc.expire import get_expired_drafts, send_expire_notice_for_draft, expire_draft
|
from ietf.doc.expire import get_expired_drafts, send_expire_notice_for_draft, expire_draft
|
||||||
from ietf.doc.factories import IndividualDraftFactory, WgDraftFactory, RgDraftFactory, DocEventFactory
|
from ietf.doc.factories import EditorialDraftFactory, IndividualDraftFactory, WgDraftFactory, RgDraftFactory, DocEventFactory
|
||||||
from ietf.doc.models import ( Document, DocReminder, DocEvent,
|
from ietf.doc.models import ( Document, DocReminder, DocEvent,
|
||||||
ConsensusDocEvent, LastCallDocEvent, RelatedDocument, State, TelechatDocEvent,
|
ConsensusDocEvent, LastCallDocEvent, RelatedDocument, State, TelechatDocEvent,
|
||||||
WriteupDocEvent, DocRelationshipName, IanaExpertDocEvent )
|
WriteupDocEvent, DocRelationshipName, IanaExpertDocEvent )
|
||||||
|
|
@ -2161,3 +2161,13 @@ class ShepherdWriteupTests(TestCase):
|
||||||
self.assertContains(r, "for Group Documents", status_code=200)
|
self.assertContains(r, "for Group Documents", status_code=200)
|
||||||
r = self.client.post(url,dict(reset_text=''))
|
r = self.client.post(url,dict(reset_text=''))
|
||||||
self.assertContains(r, "for Group Documents", status_code=200)
|
self.assertContains(r, "for Group Documents", status_code=200)
|
||||||
|
|
||||||
|
class EditorialDraftMetadataTests(TestCase):
|
||||||
|
def test_editorial_metadata(self):
|
||||||
|
draft = EditorialDraftFactory()
|
||||||
|
url = urlreverse("ietf.doc.views_doc.document_main", kwargs=dict(name=draft.name))
|
||||||
|
r = self.client.get(url)
|
||||||
|
q = PyQuery(r.content)
|
||||||
|
top_level_metadata_headings = q("tbody>tr>th:first-child").text()
|
||||||
|
self.assertNotIn("IESG", top_level_metadata_headings)
|
||||||
|
self.assertNotIn("IANA", top_level_metadata_headings)
|
||||||
|
|
|
||||||
|
|
@ -267,7 +267,7 @@
|
||||||
</tr>
|
</tr>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
</tbody>
|
</tbody>
|
||||||
{% if not doc.stream_id == 'iab' %}
|
{% if doc.stream_id != 'iab' and doc.stream_id != 'editorial' %}
|
||||||
<tbody class="meta border-top">
|
<tbody class="meta border-top">
|
||||||
<tr>
|
<tr>
|
||||||
<th scope="row">
|
<th scope="row">
|
||||||
|
|
@ -447,125 +447,127 @@
|
||||||
</tr>
|
</tr>
|
||||||
</tbody>
|
</tbody>
|
||||||
{% endif %}
|
{% endif %}
|
||||||
{% if can_edit_iana_state or iana_review_state or iana_experts_state or iana_experts_comment %}
|
{% if doc.stream_id != 'editorial' %}
|
||||||
<tbody class="meta border-top">
|
{% if can_edit_iana_state or iana_review_state or iana_experts_state or iana_experts_comment %}
|
||||||
{% if iana_review_state or can_edit_iana_state %}
|
<tbody class="meta border-top">
|
||||||
<tr>
|
{% if iana_review_state or can_edit_iana_state %}
|
||||||
<th scope="row">
|
<tr>
|
||||||
IANA
|
<th scope="row">
|
||||||
</th>
|
|
||||||
<th scope="row">
|
|
||||||
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-review" %}">
|
|
||||||
IANA review state
|
|
||||||
</a>
|
|
||||||
</th>
|
|
||||||
<td class="edit">
|
|
||||||
{% if can_edit_iana_state and not snapshot %}
|
|
||||||
<a class="btn btn-primary btn-sm"
|
|
||||||
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-review" %}">
|
|
||||||
Edit
|
|
||||||
</a>
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
<td class="{% if iana_review_state|slugify|slice:7 != "iana-ok" %}text-danger{% else %}text-success{% endif %}">
|
|
||||||
{% if not iana_review_state %}
|
|
||||||
<span class="text-body-secondary">
|
|
||||||
(None)
|
|
||||||
</span>
|
|
||||||
{% else %}
|
|
||||||
{{ iana_review_state }}
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
</tr>
|
|
||||||
{% endif %}
|
|
||||||
{% if iana_action_state or can_edit_iana_state %}
|
|
||||||
<tr>
|
|
||||||
<th scope="row">
|
|
||||||
{% if not can_edit_iana_state and not iana_review_state %}IANA{% endif %}
|
|
||||||
</th>
|
|
||||||
<th scope="row">
|
|
||||||
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-action" %}">
|
|
||||||
IANA action state
|
|
||||||
</a>
|
|
||||||
</th>
|
|
||||||
<td class="edit">
|
|
||||||
{% if can_edit_iana_state and not snapshot %}
|
|
||||||
<a class="btn btn-primary btn-sm"
|
|
||||||
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-action" %}">
|
|
||||||
Edit
|
|
||||||
</a>
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
<td>
|
|
||||||
{% if not iana_action_state %}
|
|
||||||
<span class="text-body-secondary">
|
|
||||||
(None)
|
|
||||||
</span>
|
|
||||||
{% else %}
|
|
||||||
{{ iana_action_state }}
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
</tr>
|
|
||||||
{% endif %}
|
|
||||||
{% if iana_experts_state or can_edit_iana_state %}
|
|
||||||
<tr>
|
|
||||||
<th scope="row">
|
|
||||||
{% if not can_edit_iana_state and not iana_review_state and not iana_action_state %}IANA{% endif %}
|
|
||||||
</th>
|
|
||||||
<th scope="row">
|
|
||||||
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-experts" %}">
|
|
||||||
IANA expert review state
|
|
||||||
</a>
|
|
||||||
</th>
|
|
||||||
<td class="edit">
|
|
||||||
{% if can_edit_iana_state and not snapshot %}
|
|
||||||
<a class="btn btn-primary btn-sm"
|
|
||||||
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-experts" %}">
|
|
||||||
Edit
|
|
||||||
</a>
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
<td class="{% if iana_experts_state|slugify|slice:"-2:" != "ok" %}text-danger{% else %}text-success{% endif %}">
|
|
||||||
{% if not iana_experts_state %}
|
|
||||||
<span class="text-body-secondary">
|
|
||||||
(None)
|
|
||||||
</span>
|
|
||||||
{% else %}
|
|
||||||
{{ iana_experts_state }}
|
|
||||||
{% endif %}
|
|
||||||
</td>
|
|
||||||
</tr>
|
|
||||||
{% endif %}
|
|
||||||
{% if iana_experts_comment or can_edit_iana_state %}
|
|
||||||
<tr>
|
|
||||||
<th scope="row">
|
|
||||||
{% if not can_edit_iana_state and not iana_review_state and not iana_action_state and not iana_experts_state %}
|
|
||||||
IANA
|
IANA
|
||||||
{% endif %}
|
</th>
|
||||||
</th>
|
<th scope="row">
|
||||||
<th scope="row">
|
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-review" %}">
|
||||||
IANA expert review comments
|
IANA review state
|
||||||
</th>
|
|
||||||
<td class="edit">
|
|
||||||
{% if can_edit_iana_state and not snapshot %}
|
|
||||||
<a class="btn btn-primary btn-sm"
|
|
||||||
href="{% url 'ietf.doc.views_draft.add_iana_experts_comment' name=doc.name %}">
|
|
||||||
Edit
|
|
||||||
</a>
|
</a>
|
||||||
{% endif %}
|
</th>
|
||||||
</td>
|
<td class="edit">
|
||||||
<td>
|
{% if can_edit_iana_state and not snapshot %}
|
||||||
{% if not iana_experts_comment %}
|
<a class="btn btn-primary btn-sm"
|
||||||
<span class="text-body-secondary">
|
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-review" %}">
|
||||||
(None)
|
Edit
|
||||||
</span>
|
</a>
|
||||||
{% else %}
|
{% endif %}
|
||||||
{{ iana_experts_comment }}
|
</td>
|
||||||
{% endif %}
|
<td class="{% if iana_review_state|slugify|slice:7 != "iana-ok" %}text-danger{% else %}text-success{% endif %}">
|
||||||
</td>
|
{% if not iana_review_state %}
|
||||||
</tr>
|
<span class="text-body-secondary">
|
||||||
{% endif %}
|
(None)
|
||||||
</tbody>
|
</span>
|
||||||
|
{% else %}
|
||||||
|
{{ iana_review_state }}
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
</tr>
|
||||||
|
{% endif %}
|
||||||
|
{% if iana_action_state or can_edit_iana_state %}
|
||||||
|
<tr>
|
||||||
|
<th scope="row">
|
||||||
|
{% if not can_edit_iana_state and not iana_review_state %}IANA{% endif %}
|
||||||
|
</th>
|
||||||
|
<th scope="row">
|
||||||
|
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-action" %}">
|
||||||
|
IANA action state
|
||||||
|
</a>
|
||||||
|
</th>
|
||||||
|
<td class="edit">
|
||||||
|
{% if can_edit_iana_state and not snapshot %}
|
||||||
|
<a class="btn btn-primary btn-sm"
|
||||||
|
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-action" %}">
|
||||||
|
Edit
|
||||||
|
</a>
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
<td>
|
||||||
|
{% if not iana_action_state %}
|
||||||
|
<span class="text-body-secondary">
|
||||||
|
(None)
|
||||||
|
</span>
|
||||||
|
{% else %}
|
||||||
|
{{ iana_action_state }}
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
</tr>
|
||||||
|
{% endif %}
|
||||||
|
{% if iana_experts_state or can_edit_iana_state %}
|
||||||
|
<tr>
|
||||||
|
<th scope="row">
|
||||||
|
{% if not can_edit_iana_state and not iana_review_state and not iana_action_state %}IANA{% endif %}
|
||||||
|
</th>
|
||||||
|
<th scope="row">
|
||||||
|
<a href="{% url "ietf.doc.views_help.state_help" type="draft-iana-experts" %}">
|
||||||
|
IANA expert review state
|
||||||
|
</a>
|
||||||
|
</th>
|
||||||
|
<td class="edit">
|
||||||
|
{% if can_edit_iana_state and not snapshot %}
|
||||||
|
<a class="btn btn-primary btn-sm"
|
||||||
|
href="{% url 'ietf.doc.views_draft.change_iana_state' name=doc.name state_type="iana-experts" %}">
|
||||||
|
Edit
|
||||||
|
</a>
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
<td class="{% if iana_experts_state|slugify|slice:"-2:" != "ok" %}text-danger{% else %}text-success{% endif %}">
|
||||||
|
{% if not iana_experts_state %}
|
||||||
|
<span class="text-body-secondary">
|
||||||
|
(None)
|
||||||
|
</span>
|
||||||
|
{% else %}
|
||||||
|
{{ iana_experts_state }}
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
</tr>
|
||||||
|
{% endif %}
|
||||||
|
{% if iana_experts_comment or can_edit_iana_state %}
|
||||||
|
<tr>
|
||||||
|
<th scope="row">
|
||||||
|
{% if not can_edit_iana_state and not iana_review_state and not iana_action_state and not iana_experts_state %}
|
||||||
|
IANA
|
||||||
|
{% endif %}
|
||||||
|
</th>
|
||||||
|
<th scope="row">
|
||||||
|
IANA expert review comments
|
||||||
|
</th>
|
||||||
|
<td class="edit">
|
||||||
|
{% if can_edit_iana_state and not snapshot %}
|
||||||
|
<a class="btn btn-primary btn-sm"
|
||||||
|
href="{% url 'ietf.doc.views_draft.add_iana_experts_comment' name=doc.name %}">
|
||||||
|
Edit
|
||||||
|
</a>
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
<td>
|
||||||
|
{% if not iana_experts_comment %}
|
||||||
|
<span class="text-body-secondary">
|
||||||
|
(None)
|
||||||
|
</span>
|
||||||
|
{% else %}
|
||||||
|
{{ iana_experts_comment }}
|
||||||
|
{% endif %}
|
||||||
|
</td>
|
||||||
|
</tr>
|
||||||
|
{% endif %}
|
||||||
|
</tbody>
|
||||||
|
{% endif %}
|
||||||
{% endif %}
|
{% endif %}
|
||||||
{% if rfc_editor_state %}
|
{% if rfc_editor_state %}
|
||||||
<tbody class="meta border-top">
|
<tbody class="meta border-top">
|
||||||
|
|
|
||||||
Loading…
Reference in a new issue