Skip to content

Commit

Permalink
Include last call review-end-date in the rendered summary (#2918)
Browse files Browse the repository at this point in the history
* Include last call review-end-date in the rendered summary

* Split review-period-ends line into its own on the EIP render
  • Loading branch information
axic authored Sep 3, 2020
1 parent d90e668 commit 6fce40f
Show file tree
Hide file tree
Showing 2 changed files with 10 additions and 2 deletions.
10 changes: 9 additions & 1 deletion _includes/eiptable.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,19 @@
<h2 id="{{status|slugify}}">{{status}}</h2>
<table class="eiptable">
<thead>
<tr><th class="eipnum">Number</th><th class="title">Title</th><th class="author">Author</th></tr>
{% if status == "Last Call" %}
<tr>
<th class="eipnum">Number</th><th class="date">Review ends</th><th class="title">Title</th><th class="author">Author</th></tr>
{% else %}
<tr><th class="eipnum">Number</th><th class="title">Title</th><th class="author">Author</th></tr>
{% endif %}
</thead>
{% for page in eips %}
<tr>
<td class="eipnum"><a href="{{page.url|relative_url}}">{{page.eip|xml_escape}}</a></td>
{% if status == "Last Call" and page.review-period-end != undefined %}
<td class="date">{{ page.review-period-end | xml_escape }}</td>
{% endif %}
<td class="title">{{page.title|xml_escape}}</td>
<td class="author">{% include authorlist.html authors=page.author %}</td>
</tr>
Expand Down
2 changes: 1 addition & 1 deletion _layouts/eip.html
Original file line number Diff line number Diff line change
Expand Up @@ -14,7 +14,7 @@ <h1 class="page-heading">
{% endif %}
<tr><th>Status</th><td>{{ page.status | xml_escape }}
{% if page.review-period-end != undefined %}
<strong>(review ends {{ page.review-period-end | xml_escape }})</strong>
<tr><th>Review period ends</th><td>{{ page.review-period-end | xml_escape }}</td></tr>
{% endif %}
</td></tr>
<tr><th>Type</th><td>{{ page.type | xml_escape }}</td></tr>
Expand Down

0 comments on commit 6fce40f

Please sign in to comment.