feat: Add PeerReviewer model for improved peer review support

django/core/jinja2/core/member_profiles/retrieve.jinja

@@ -158,8 +158,7 @@
                     <p class='lead p-3'>
                     You haven't been invited to review other CoMSES member's models yet. If you would like to be added
                     to the pool of CoMSES Computational Model Library 
-                    <a href='{{url("library:peer-review-overview")}}'>Peer Reviewers</a> please 
-                    <a href='{{slugurl("contact")}}'>let us know</a>.
+                    <a href='{{url("library:peer-review-overview")}}'>Peer Reviewers</a> please visit your <a href='{{ profile.get_edit_url() }}'>profile edit page</a> and create a peer reviewer profile.
                     </p>
                 {% endif %}
             </div>

django/core/models.py

@@ -42,7 +42,6 @@ class ComsesGroups(Enum):
     MODERATOR = "Moderators"
     EDITOR = "Editors"
     FULL_MEMBER = "Full Members"
-    REVIEWER = "Reviewers"
 
     @staticmethod
     @transaction.atomic
@@ -437,6 +436,10 @@ def discourse_username(self):
     def is_active(self):
         return self.user.is_active
 
+    @property
+    def is_reviewer(self):
+        return hasattr(self, "peer_reviewer")
+
     # Urls
     @property
     def orcid_url(self):
@@ -510,10 +513,6 @@ def primary_affiliation_name(self):
     def submitter(self):
         return self.user
 
-    @cached_property
-    def is_reviewer(self):
-        return ComsesGroups.REVIEWER.is_member(self.user)
-
     @cached_property
     def name(self):
         return self.user.get_full_name() or self.user.username

django/core/serializers.py

@@ -144,6 +144,9 @@ class MemberProfileSerializer(serializers.ModelSerializer):
     profile_url = serializers.URLField(source="get_absolute_url", read_only=True)
     bio = MarkdownField()
     research_interests = MarkdownField()
+    peer_reviewer_id = serializers.IntegerField(
+        source="peer_reviewer.id", read_only=True, allow_null=True, default=None
+    )
 
     def validate_affiliations(self, value):
         return validate_affiliations(value)
@@ -240,6 +243,7 @@ class Meta:
         model = MemberProfile
         fields = (
             # User
+            "id",
             "date_joined",
             "family_name",
             "given_name",
@@ -261,12 +265,12 @@ class Meta:
             "orcid_url",
             "github_url",
             "personal_url",
-            "is_reviewer",
             "professional_url",
             "profile_url",
             "research_interests",
             "affiliations",
             "name",
+            "peer_reviewer_id",
         )
 
 

django/library/forms.py

@@ -27,6 +27,7 @@ class Meta:
             "review",
             "editor",
             "candidate_reviewer",
+            "reviewer",
         ]
 
 

django/library/jinja2/library/review/dashboard.jinja

@@ -1,10 +1,26 @@
 {% extends 'sidebar_layout.jinja' %}
+{% from "common.jinja" import breadcrumb %}
 {% from "library/review/includes/macros.jinja" import display_closed_status %}
 
 {% block introduction %}
     <h1>Peer Review Editor Dashboard</h1>
 {% endblock %}
 
+{% block top %}
+    {{ breadcrumb([
+        {'url': '/reviews/', 'text': 'Reviews'},
+        {'text': 'Review Editor Dashboard' }])
+    }}
+    <ul class="nav nav-tabs justify-content-center mb-4">
+        <li class="nav-item">
+            <a href="/reviews/dashboard" class="nav-link active">Reviews</a>
+        </li>
+        <li class="nav-item">
+            <a href="/reviews/reviewers" class="nav-link">Reviewers</a>
+        </li>
+    </ul>
+{% endblock %}
+
 {% block content %}
     {% for codebase in codebases %}
         <div class="card">

django/library/jinja2/library/review/email/review_invitation_declined.jinja

@@ -1,6 +1,6 @@
 Dear {{ invitation.invitee }},
 
-Thank you for letting us know you were unable to accept the invitation to provide peer review for this model. If you no longer wish to serve as a CoMSES Net peer reviewer please [let us know]({{build_absolute_uri(slugurl("contact"))}}) and we will remove you from this opt-in group. *Note:* submitting your own computational model for peer review automatically adds you to the pool of candidate peer reviewers.
+Thank you for letting us know you were unable to accept the invitation to provide peer review for this model. If you no longer wish to serve as a CoMSES Net peer reviewer please visit your <a href='{{ profile.get_edit_url() }}'>profile edit page</a> and deactivate your peer reviewer profile.
 
 Best regards,
 

django/library/jinja2/library/review/email/reviewer_submitted.jinja

@@ -1,3 +1,3 @@
 Dear {{invitation.editor.name}},
 
-The reviewer {{ invitation.candidate_reviewer }} has submitted their feedback. Action is needed on your part to continue the peer review process and send the recommendation back to the original submitter. You can do this at the [review management page]({{ build_absolute_uri(review.get_absolute_url()) }}).
+The reviewer {{ invitation.reviewer.member_profile }} has submitted their feedback. Action is needed on your part to continue the peer review process and send the recommendation back to the original submitter. You can do this at the [review management page]({{ build_absolute_uri(review.get_absolute_url()) }}).

django/library/jinja2/library/review/feedback/editor_update.jinja

@@ -8,7 +8,7 @@
 {% block content %}
     <h2>Editor Peer Review Management</h2>
     <p class='lead'>
-    Feedback submitted by <mark>{{review_feedback.invitation.candidate_reviewer}}</mark> on {{format_datetime(review_feedback.last_modified)}}.
+    Feedback submitted by <mark>{{review_feedback.invitation.reviewer.member_profile}}</mark> on {{format_datetime(review_feedback.last_modified)}}.
     </p>
     <h2>{{ review_feedback.invitation.review.title }}<a href="{{ review_feedback.invitation.review.codebase_release.share_url }}">(click to view in a new window/tab)</a>
     </h2>

django/library/jinja2/library/review/reviewers.jinja

@@ -0,0 +1,26 @@
+{% extends 'base.jinja' %}
+{% from "common.jinja" import breadcrumb %}
+
+{% block introduction %}
+    <h1>Manage Peer Reviewers</h1>
+{% endblock %}
+
+{% block content %}
+    {{ breadcrumb([
+        {'url': '/reviews/', 'text': 'Reviews'},
+        {'text': 'Manage Reviewers' }])
+    }}
+    <ul class="nav nav-tabs justify-content-center mb-4">
+        <li class="nav-item">
+            <a href="/reviews/dashboard" class="nav-link">Reviews</a>
+        </li>
+        <li class="nav-item">
+            <a href="/reviews/reviewers" class="nav-link active">Reviewers</a>
+        </li>
+    </ul>
+    <div id="reviewer-list"></div>
+{% endblock %}
+
+{% block js %}
+    {{ vite_asset("apps/reviewer_list.ts") }}
+{% endblock %}

django/library/migrations/0029_peerreviewer.py

@@ -0,0 +1,66 @@
+# Generated by Django 4.2.14 on 2024-07-11 17:58
+
+import django.contrib.postgres.fields
+from django.db import migrations, models
+import django.db.models.deletion
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("core", "0021_add_spam_moderation"),
+        ("library", "0028_contributor_json_affiliations"),
+    ]
+
+    operations = [
+        migrations.CreateModel(
+            name="PeerReviewer",
+            fields=[
+                (
+                    "id",
+                    models.AutoField(
+                        auto_created=True,
+                        primary_key=True,
+                        serialize=False,
+                        verbose_name="ID",
+                    ),
+                ),
+                ("date_created", models.DateTimeField(auto_now_add=True)),
+                ("is_active", models.BooleanField(default=True)),
+                (
+                    "programming_languages",
+                    django.contrib.postgres.fields.ArrayField(
+                        base_field=models.CharField(max_length=100),
+                        blank=True,
+                        default=list,
+                        help_text="Programming Languages this reviewer knows, e.g. Python, Java",
+                        size=None,
+                    ),
+                ),
+                (
+                    "subject_areas",
+                    django.contrib.postgres.fields.ArrayField(
+                        base_field=models.CharField(max_length=100),
+                        blank=True,
+                        default=list,
+                        help_text="Areas of expertise, e.g. social science, biology",
+                        size=None,
+                    ),
+                ),
+                (
+                    "notes",
+                    models.TextField(
+                        blank=True, help_text="Any additional notes about this reviewer"
+                    ),
+                ),
+                (
+                    "member_profile",
+                    models.OneToOneField(
+                        on_delete=django.db.models.deletion.CASCADE,
+                        related_name="peer_reviewer",
+                        to="core.memberprofile",
+                    ),
+                ),
+            ],
+        ),
+    ]

django/library/migrations/0030_peerreviewinvitation.py

@@ -0,0 +1,52 @@
+# Generated by Django 4.2.14 on 2024-09-20 23:12
+
+from django.db import migrations, models
+import django.db.models.deletion
+from django.utils import timezone
+from datetime import timedelta
+
+
+def create_reviewers(apps, schema):
+    PeerReviewInvitation = apps.get_model("library", "PeerReviewInvitation")
+    PeerReviewer = apps.get_model("library", "PeerReviewer")
+    PeerReviewerFeedback = apps.get_model("library", "PeerReviewerFeedback")
+
+    one_year_ago = timezone.now() - timedelta(days=365)
+    for invitation in PeerReviewInvitation.objects.all():
+        member_profile = invitation.candidate_reviewer
+        reviewer, created = PeerReviewer.objects.get_or_create(
+            member_profile=member_profile,
+            defaults={
+                "is_active": PeerReviewerFeedback.objects.filter(
+                    invitation__candidate_reviewer=member_profile,
+                    last_modified__gte=one_year_ago,
+                ).exists()
+            },
+        )
+        invitation.reviewer = reviewer
+        invitation.save()
+
+
+class Migration(migrations.Migration):
+
+    dependencies = [
+        ("library", "0029_peerreviewer"),
+    ]
+
+    operations = [
+        migrations.AddField(
+            model_name="peerreviewinvitation",
+            name="reviewer",
+            field=models.ForeignKey(
+                null=True,
+                on_delete=django.db.models.deletion.PROTECT,
+                related_name="invitation_set",
+                to="library.peerreviewer",
+            ),
+        ),
+        migrations.RunPython(create_reviewers),
+        migrations.AlterUniqueTogether(
+            name="peerreviewinvitation",
+            unique_together={("review", "reviewer")},
+        ),
+    ]