Skip to content

Commit 79212ba

Browse files
committed
Explicitly determine approving body and consensus body in publication
requests for alternate streams so that they appear correctly for the IRTF - Legacy-Id: 4897
1 parent 400218f commit 79212ba

2 files changed

Lines changed: 12 additions & 3 deletions

File tree

ietf/idrfc/mails.py

Lines changed: 10 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -291,11 +291,20 @@ def generate_publication_request(request, doc):
291291

292292
e = doc.latest_event(ConsensusDocEvent, type="changed_consensus")
293293
consensus = e.consensus if e else None
294-
294+
295+
if doc.stream_id == "irtf":
296+
approving_body = "IRSG"
297+
consensus_body = doc.group.acronym.upper()
298+
else:
299+
approving_body = str(doc.stream)
300+
consensus_body = approving_body
301+
295302
return render_to_string("idrfc/publication_request.txt",
296303
dict(doc=doc,
297304
doc_url=settings.IDTRACKER_BASE_URL + doc.get_absolute_url(),
298305
group_description=group_description,
306+
approving_body=approving_body,
307+
consensus_body=consensus_body,
299308
consensus=consensus,
300309
)
301310
)

ietf/templates/idrfc/publication_request.txt

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,13 +1,13 @@
11
{% load mail_filters %}{% autoescape off %}{% filter wordwrap:73 %}
2-
The {{ doc.stream }} has approved the following document:
2+
The {{ approving_body }} has approved the following document in the {{ doc.stream }} stream:
33

44
- "{{ doc.title }}" ({{ doc.name }}) as {{ doc|std_level_prompt }}.{% if group_description %}
55

66
This document is the product of the {{ group_description }}.{% endif %}{% endfilter %}
77

88
URL: {{ doc_url }}
99

10-
{% filter wordwrap:73 %}{% if consensus != None %}The document {% if consensus %}represents{% else %}does not necessarily represent{% endif%} the consensus of the {{ doc.stream }}.
10+
{% filter wordwrap:73 %}{% if consensus != None %}The document {% if consensus %}represents{% else %}does not necessarily represent{% endif%} the consensus of the {{ consensus_body }}.
1111

1212
{% endif %}No IANA allocation in the document requires IETF Consensus or Standards Action.{% endfilter %}
1313

0 commit comments

Comments
 (0)