add organizers field to contest serialization (#146)

fix: add organizers field to contest serialization

- Add organizers to Contest.to_dict() method to include them in API
responses
- Fixes missing organizer names in contest list and detail views
- Organizers list now correctly shows creator + additional organizers
- Resolves frontend display issue where only creator was visible

Author: GauriGupta21

backend/alembic/versions/9ce4fca210b6_check_for_model_changes.py

@@ -5,13 +5,16 @@
 Create Date: 2026-01-13 20:39:43.150557
 
 """
+
 from alembic import op
 import sqlalchemy as sa
 from sqlalchemy.dialects import mysql
+from sqlalchemy import inspect
+
 
 # revision identifiers, used by Alembic.
-revision = '9ce4fca210b6'
-down_revision = '30c4f4dbcb85'
+revision = "9ce4fca210b6"
+down_revision = "30c4f4dbcb85"
 branch_labels = None
 depends_on = None
 
@@ -20,64 +23,99 @@ def upgrade() -> None:
     # ### commands auto generated by Alembic - please adjust! ###
     # Change scoring_parameters from JSON to Text (to match model definition)
     # Keep the comment as it provides useful documentation
-    op.alter_column('contests', 'scoring_parameters',
-               existing_type=mysql.JSON(),
-               type_=sa.Text(),
-               comment='Weights for scoring parameters',
-               existing_comment='Weights for scoring parameters',
-               existing_nullable=True)
+    op.alter_column(
+        "contests",
+        "scoring_parameters",
+        existing_type=mysql.JSON(),
+        type_=sa.Text(),
+        comment="Weights for scoring parameters",
+        existing_comment="Weights for scoring parameters",
+        existing_nullable=True,
+    )
     # Keep server default and comment for min_reference_count
-    op.alter_column('contests', 'min_reference_count',
-               existing_type=mysql.INTEGER(),
-               server_default=sa.text("'0'"),
-               comment='Minimum number of references required for article submissions',
-               existing_comment='Minimum number of references required for article submissions',
-               existing_nullable=False)
+    op.alter_column(
+        "contests",
+        "min_reference_count",
+        existing_type=mysql.INTEGER(),
+        server_default=sa.text("'0'"),
+        comment="Minimum number of references required for article submissions",
+        existing_comment="Minimum number of references required for article submissions",
+        existing_nullable=False,
+    )
     # Keep comment for organizers
-    op.alter_column('contests', 'organizers',
-               existing_type=mysql.TEXT(),
-               comment='Comma-separated list of organizer usernames',
-               existing_comment='Comma-separated list of organizer usernames',
-               existing_nullable=True)
+    op.alter_column(
+        "contests",
+        "organizers",
+        existing_type=mysql.TEXT(),
+        comment="Comma-separated list of organizer usernames",
+        existing_comment="Comma-separated list of organizer usernames",
+        existing_nullable=True,
+    )
     # Change parameter_scores from JSON to Text (to match model definition)
     # Keep the comment as it provides useful documentation
-    op.alter_column('submissions', 'parameter_scores',
-               existing_type=mysql.JSON(),
-               type_=sa.Text(),
-               comment='Per-parameter jury scores and comments',
-               existing_comment='Per-parameter jury scores and comments',
-               existing_nullable=True)
+    op.alter_column(
+        "submissions",
+        "parameter_scores",
+        existing_type=mysql.JSON(),
+        type_=sa.Text(),
+        comment="Per-parameter jury scores and comments",
+        existing_comment="Per-parameter jury scores and comments",
+        existing_nullable=True,
+    )
     # Drop index that was detected as removed from model
-    op.drop_index(op.f('idx_user_contest_submission'), table_name='submissions')
+    # Drop index that was detected as removed from model (MySQL-safe)
+    conn = op.get_bind()
+    inspector = inspect(conn)
+
+    indexes = [idx["name"] for idx in inspector.get_indexes("submissions")]
+
+    if "idx_user_contest_submission" in indexes:
+        op.drop_index("idx_user_contest_submission", table_name="submissions")
     # ### end Alembic commands ###
 
 
 def downgrade() -> None:
     # ### commands auto generated by Alembic - please adjust! ###
     # Recreate index that was dropped in upgrade
-    op.create_index(op.f('idx_user_contest_submission'), 'submissions', ['user_id', 'contest_id'], unique=False)
+    op.create_index(
+        op.f("idx_user_contest_submission"),
+        "submissions",
+        ["user_id", "contest_id"],
+        unique=False,
+    )
     # Revert parameter_scores back to JSON type
-    op.alter_column('submissions', 'parameter_scores',
-               existing_type=sa.Text(),
-               type_=mysql.JSON(),
-               comment='Per-parameter jury scores and comments',
-               existing_nullable=True)
+    op.alter_column(
+        "submissions",
+        "parameter_scores",
+        existing_type=sa.Text(),
+        type_=mysql.JSON(),
+        comment="Per-parameter jury scores and comments",
+        existing_nullable=True,
+    )
     # Revert organizers (no change needed, just keeping comment)
-    op.alter_column('contests', 'organizers',
-               existing_type=mysql.TEXT(),
-               comment='Comma-separated list of organizer usernames',
-               existing_nullable=True)
+    op.alter_column(
+        "contests",
+        "organizers",
+        existing_type=mysql.TEXT(),
+        comment="Comma-separated list of organizer usernames",
+        existing_nullable=True,
+    )
     # Revert min_reference_count (keep default and comment)
-    op.alter_column('contests', 'min_reference_count',
-               existing_type=mysql.INTEGER(),
-               server_default=sa.text("'0'"),
-               comment='Minimum number of references required for article submissions',
-               existing_nullable=False)
+    op.alter_column(
+        "contests",
+        "min_reference_count",
+        existing_type=mysql.INTEGER(),
+        server_default=sa.text("'0'"),
+        comment="Minimum number of references required for article submissions",
+        existing_nullable=False,
+    )
     # Revert scoring_parameters back to JSON type
-    op.alter_column('contests', 'scoring_parameters',
-               existing_type=sa.Text(),
-               type_=mysql.JSON(),
-               comment='Weights for scoring parameters',
-               existing_nullable=True)
+    op.alter_column(
+        "contests",
+        "scoring_parameters",
+        existing_type=sa.Text(),
+        type_=mysql.JSON(),
+        comment="Weights for scoring parameters",
+        existing_nullable=True,
+    )
     # ### end Alembic commands ###
-

backend/app/models/contest.py

@@ -732,6 +732,7 @@ def to_dict(self):
             'min_reference_count': self.min_reference_count,  # Minimum reference count requirement
             'categories': self.get_categories(),
             'jury_members': self.get_jury_members(),
+            'organizers': self.get_organizers(),
             'template_link': self.template_link,  # Template link for contest (optional)
             # Format datetime as ISO string with 'Z' suffix to indicate UTC
             # This ensures JavaScript interprets it as UTC, not local time