rename BoardOfDirectors.reviewers to claim_reviewers

Signed-off-by: Rudransh Shrivastava <rudransh.shrivastava@owasp.org>

Author: rudransh-shrivastava

backend/src/apps/owasp/admin/board_of_directors.py

@@ -8,7 +8,7 @@
 class BoardOfDirectorsAdmin(admin.ModelAdmin):
     """Admin for Snapshot model."""
 
-    filter_horizontal = ("reviewers",)
+    filter_horizontal = ("claim_reviewers",)
     list_filter = ("year",)
     ordering = ("-year",)
     search_fields = ("year",)

backend/src/apps/owasp/api/internal/mutations/board_candidate_claim_review.py

@@ -87,7 +87,7 @@ def create_board_candidate_claim_review(
         user = info.context.request.user
 
         is_reviewer = BoardOfDirectors.objects.filter(
-            year=input_data.year, reviewers=user
+            year=input_data.year, claim_reviewers=user
         ).exists()
         if not user.github_user or not is_reviewer:
             return ReviewResult(ok=False, code="FORBIDDEN", message=ACCESS_DENIED_MSG)

backend/src/apps/owasp/api/internal/nodes/board_candidate_claim.py

@@ -60,7 +60,9 @@ def reviews(
         if is_self or root.status == BoardCandidateClaim.Status.APPROVED:
             return root.reviews.all()
 
-        is_reviewer = user.is_authenticated and root.board.reviewers.filter(id=user.id).exists()
+        is_reviewer = (
+            user.is_authenticated and root.board.claim_reviewers.filter(id=user.id).exists()
+        )
         if is_reviewer:
             return root.reviews.filter(reviewer=user)
         return []

backend/src/apps/owasp/api/internal/nodes/board_of_directors.py

@@ -43,6 +43,8 @@ def owasp_url(self, root: BoardOfDirectors) -> str:
     def reviewer(self, root: BoardOfDirectors, login: str) -> UserNode | None:
         """Resolve board election reviewer."""
         user = (
-            root.reviewers.select_related("github_user").filter(github_user__login=login).first()
+            root.claim_reviewers.select_related("github_user")
+            .filter(github_user__login=login)
+            .first()
         )
         return user.github_user if user else None

backend/src/apps/owasp/api/internal/queries/board_candidate_claim.py

@@ -32,7 +32,7 @@ def board_candidate_claims(
         user = info.context.request.user
         is_reviewer = (
             user.is_authenticated
-            and BoardOfDirectors.objects.filter(year=year, reviewers=user).exists()
+            and BoardOfDirectors.objects.filter(year=year, claim_reviewers=user).exists()
         )
         claims = BoardCandidateClaim.objects.filter(
             board__year=year,
@@ -126,7 +126,9 @@ def board_candidate_claim(
             and user.github_user is not None
             and user.github_user == claim.candidate.member
         )
-        is_reviewer = user.is_authenticated and claim.board.reviewers.filter(id=user.id).exists()
+        is_reviewer = (
+            user.is_authenticated and claim.board.claim_reviewers.filter(id=user.id).exists()
+        )
 
         return (
             claim

backend/src/apps/owasp/api/internal/queries/board_candidate_claim_evidence.py

@@ -44,7 +44,7 @@ def get_claim_evidence(
         and user.github_user == evidence.claim.candidate.member
     )
     is_reviewer = (
-        user.is_authenticated and evidence.claim.board.reviewers.filter(id=user.id).exists()
+        user.is_authenticated and evidence.claim.board.claim_reviewers.filter(id=user.id).exists()
     )
 
     return (
@@ -92,7 +92,9 @@ def board_candidate_claim_evidences(
             and claim.candidate.member is not None
             and user.github_user == claim.candidate.member
         )
-        is_reviewer = user.is_authenticated and claim.board.reviewers.filter(id=user.id).exists()
+        is_reviewer = (
+            user.is_authenticated and claim.board.claim_reviewers.filter(id=user.id).exists()
+        )
 
         return (
             claim.evidences.filter(is_removed=False)

backend/src/apps/owasp/migrations/0082_remove_boardofdirectors_reviewers_and_more.py

@@ -0,0 +1,29 @@
+# Generated by Django 6.0.6 on 2026-08-08 15:51
+
+from django.conf import settings
+from django.db import migrations, models
+
+
+class Migration(migrations.Migration):
+    dependencies = [
+        ("owasp", "0081_alter_boardcandidateclaim_board_and_more"),
+        migrations.swappable_dependency(settings.AUTH_USER_MODEL),
+    ]
+
+    operations = [
+        migrations.RemoveField(
+            model_name="boardofdirectors",
+            name="reviewers",
+        ),
+        migrations.AddField(
+            model_name="boardofdirectors",
+            name="claim_reviewers",
+            field=models.ManyToManyField(
+                blank=True,
+                help_text="Reviewers for this year's board election claims.",
+                related_name="+",
+                to=settings.AUTH_USER_MODEL,
+                verbose_name="Claim reviewers",
+            ),
+        ),
+    ]

backend/src/apps/owasp/models/board_candidate_claim_review.py

@@ -64,7 +64,7 @@ def clean(self) -> None:
             err = "Review can only be added to submitted claims."
             raise ValidationError(err)
 
-        if not self.claim.board.reviewers.filter(id=self.reviewer.id).exists():
+        if not self.claim.board.claim_reviewers.filter(id=self.reviewer.id).exists():
             err = "Only Claim Reviewers can review claims."
             raise ValidationError(err)
 

backend/src/apps/owasp/models/board_of_directors.py

@@ -30,9 +30,9 @@ class Meta:
 
     created_at = models.DateTimeField(auto_now_add=True)
     updated_at = models.DateTimeField(auto_now=True)
-    reviewers = models.ManyToManyField(
+    claim_reviewers = models.ManyToManyField(
         "nest.User",
-        verbose_name="Reviewers",
+        verbose_name="Claim reviewers",
         related_name="+",
         blank=True,
         help_text="Reviewers for this year's board election claims.",

backend/tests/unit/apps/owasp/admin/board_of_directors_test.py

@@ -11,10 +11,10 @@ class TestBoardOfDirectorsAdmin:
     """Tests for BoardOfDirectorsAdmin."""
 
     def test_filter_horizontal(self):
-        """Test filter_horizontal includes reviewers."""
+        """Test filter_horizontal includes claim_reviewers."""
         admin_instance = BoardOfDirectorsAdmin(BoardOfDirectors, AdminSite())
 
-        assert admin_instance.filter_horizontal == ("reviewers",)
+        assert admin_instance.filter_horizontal == ("claim_reviewers",)
 
     def test_list_filter(self):
         """Test list_filter includes year."""