From b82919b8c9b99fd046135d4dc6ed1965c2fa2c23 Mon Sep 17 00:00:00 2001
From: Jon Wood <jon@blankpad.net>
Date: Sat, 25 May 2024 17:51:49 +0000
Subject: [PATCH] Changelog page to review all proposal changes

---
 apps/cfp_review/base.py                     | 12 +++++
 templates/cfp_review/_nav.html              |  1 +
 templates/cfp_review/proposal_versions.html | 50 +++++++++++++++++++++
 3 files changed, 63 insertions(+)
 create mode 100644 templates/cfp_review/proposal_versions.html

diff --git a/apps/cfp_review/base.py b/apps/cfp_review/base.py
index 6eae779c7..a06529361 100644
--- a/apps/cfp_review/base.py
+++ b/apps/cfp_review/base.py
@@ -24,6 +24,7 @@
 from models.permission import Permission
 from sqlalchemy import func, exists, select
 from sqlalchemy.orm import joinedload, undefer
+from sqlalchemy_continuum.utils import version_class
 
 from main import db, external_url
 from .estimation import get_cfp_estimate
@@ -668,6 +669,17 @@ def message_proposer(proposal_id):
     )
 
 
+@cfp_review.route("/proposals/versions")
+@admin_required
+def proposal_versions():
+    size = int(request.args.get("size", "100"))
+    version_cls = version_class(Proposal)
+    versions = version_cls.query.order_by(version_cls.modified.desc())
+    paged_versions = db.paginate(versions, per_page=size, error_out=False)
+
+    return render_template("cfp_review/proposal_versions.html", versions=paged_versions)
+
+
 @cfp_review.route("/proposals/<int:proposal_id>/versions")
 @admin_required
 def proposal_latest_version(proposal_id):
diff --git a/templates/cfp_review/_nav.html b/templates/cfp_review/_nav.html
index 778e47055..814f21260 100644
--- a/templates/cfp_review/_nav.html
+++ b/templates/cfp_review/_nav.html
@@ -33,6 +33,7 @@
                 {{ menuitem('Manual Review', ".proposals", proposal_counts['manual-review'], state='manual-review')}}
                 <li role="separator" class="divider"></li>
                 {{ menuitem('Summary', ".proposals_summary")}}
+                {{ menuitem('Changelog', ".proposal_versions" )}}
             </ul>
         </li>
         <li class="dropdown">
diff --git a/templates/cfp_review/proposal_versions.html b/templates/cfp_review/proposal_versions.html
new file mode 100644
index 000000000..ed5a2938f
--- /dev/null
+++ b/templates/cfp_review/proposal_versions.html
@@ -0,0 +1,50 @@
+{% from "_formhelpers.html" import render_field %}
+{% extends "cfp_review/base.html" %}
+{% block body %}
+<table class="table table-condensed">
+  <tr>
+    <th>ID</th>
+    <th>Type</th>
+    <th>Title</th>
+    <th>Modified</th>
+    <th>User</th>
+  </tr>
+  {% for version in versions %}
+    <tr>
+      <td>{{ version.id }}-{{ version.transaction_id }}</td>
+      <td>{{ version.type }}</td>
+      <td>
+        <a href="{{ url_for('.proposal_version', proposal_id=version.id, txn_id=version.transaction_id) }}">
+          {{ version.title }}
+        </a>
+      </td>
+      <td>{{ version.modified.strftime("%Y-%m-%d %H:%M") }}</td>
+      <td>{{ version.user.name }} ({{ version.user.email }}</td>
+    </tr>
+  {% endfor %}
+</table>
+
+{% if versions.pages > 1 %}
+<div>
+  {{ versions.first }} - {{ versions.last }} of {{ versions.total }}
+</div>
+
+<nav aria-label="Version navigation">
+  <ul class="pagination">
+    {% if versions.has_prev %}
+    <li class="page-item"><a class="page-link" href="{{url_for('.proposal_versions', page=versions.prev_num, size=versions.per_page )}}">
+      <span aria-hidden="true">&laquo;</span>
+      <span class="sr-only">Previous</span>
+    </a></li>
+    {% endif %}
+    <li class="page-item active"><a class="page-link" href="#">{{ versions.page }}</a></li>
+    {% if versions.has_next %}
+    <li class="page-item"><a class="page-link" href="{{url_for('.proposal_versions', page=versions.next_num, size=versions.per_page)}}">
+      <span aria-hidden="true">&raquo;</span>
+      <span class="sr-only">Next</span>
+    </a></li>
+    {% endif %}
+  </ul>
+</nav>
+{% endif %}
+{% endblock %}