From 098a23a2000f8df0eb12b6471cc02679410b29f6 Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Mon, 9 Mar 2026 12:36:41 +0100
Subject: [PATCH 1/6] submission pagination banckend ok

---
 src/apps/api/views/submissions.py | 2 ++
 1 file changed, 2 insertions(+)

diff --git a/src/apps/api/views/submissions.py b/src/apps/api/views/submissions.py
index 7c7ddb538..2d95e2e06 100644
--- a/src/apps/api/views/submissions.py
+++ b/src/apps/api/views/submissions.py
@@ -16,6 +16,7 @@
 from django.core.files.base import ContentFile
 
 from profiles.models import Organization, Membership
+from api.pagination import LargePagination
 from tasks.models import Task
 from api.serializers.submissions import SubmissionCreationSerializer, SubmissionSerializer, SubmissionFilesSerializer, SubmissionDetailSerializer
 from competitions.models import Submission, SubmissionDetails, Phase, CompetitionParticipant
@@ -32,6 +33,7 @@ class SubmissionViewSet(ModelViewSet):
     filter_fields = ('phase__competition', 'phase', 'status', 'is_soft_deleted')
     search_fields = ('data__data_file', 'description', 'name', 'owner__username')
     renderer_classes = api_settings.DEFAULT_RENDERER_CLASSES + [renderers.CSVRenderer]
+    pagination_class = LargePagination
 
     def check_object_permissions(self, request, obj):
         if self.action in ['submission_leaderboard_connection']:

From 90c95b9048ff77d4d04005c6b5166127e36158d0 Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Mon, 9 Mar 2026 13:09:36 +0100
Subject: [PATCH 2/6] UI for submisiions pagniation OK, feature complete

---
 .../detail/submission_manager.tag             | 115 ++++++++++++++++--
 1 file changed, 104 insertions(+), 11 deletions(-)

diff --git a/src/static/riot/competitions/detail/submission_manager.tag b/src/static/riot/competitions/detail/submission_manager.tag
index 5ffe959ef..3eeeb034f 100644
--- a/src/static/riot/competitions/detail/submission_manager.tag
+++ b/src/static/riot/competitions/detail/submission_manager.tag
@@ -187,6 +187,37 @@
         </tbody>
     </table>
 
+    <div class="ui pagination menu" style="display:flex; align-items:center; justify-content:space-between; margin-top: 12px;">
+        <div style="display:flex; align-items:center; gap:8px;">
+            <button class="ui button" onclick="{ go_to_page.bind(this, page - 1) }" disabled="{ page <= 1 }">
+                <i class="icon chevron left"></i> Previous
+            </button>
+
+            <div style="display:flex; align-items:center; gap:6px;">
+                <span>Page</span>
+                <input type="number" min="1" value="{ page }" onkeydown="{ handle_page_enter }" style="width:70px; text-align:center;" />
+                <span> / { total_pages || 1 }</span>
+            </div>
+
+            <button class="ui button" onclick="{ go_to_page.bind(this, page + 1) }" disabled="{ !next }">
+                Next <i class="icon chevron right"></i>
+            </button>
+        </div>
+
+        <div style="display:flex; align-items:center; gap:8px;">
+            <label>Per page</label>
+            <select class="ui dropdown" value="{ page_size }" onchange="{ change_page_size.bind(this) }">
+                <option value="5">5</option>
+                <option value="10">10</option>
+                <option value="20">20</option>
+                <option value="50">50</option>
+            </select>
+
+            <div style="margin-left:10px; color: #8c8c8c;">
+                <small>{ total_count || 0 } total</small>
+            </div>
+    </div>
+
     <div class="ui large modal" ref="modal">
         <div class="content">
             <div if="{!!selected_submission && !_.get(selected_submission, 'has_children', false)}">
@@ -234,6 +265,13 @@
         self.checked_submissions = []
         self.show_is_soft_deleted = false
 
+        self.page = 1
+        self.page_size = 20
+        self.total_count = 0
+        self.total_pages = 1
+        self.next = null
+        self.previous = null
+
         self.on("mount", function () {
             $(self.refs.search).dropdown()
             $(self.refs.status).dropdown()
@@ -269,37 +307,63 @@
             self.update()
         }
 
+        
         self.update_submissions = function (filters) {
             self.loading = true
             self.update()
-            if (opts.admin) {
-                filters = filters || { phase__competition: opts.competition.id }
-                filters.show_is_soft_deleted = self.show_is_soft_deleted
-            } else {
-                filters = filters || { phase: self.selected_phase.id }
+
+            if (!filters) {
+                if (opts.admin) {
+                    filters = { phase__competition: opts.competition.id }
+                    filters.show_is_soft_deleted = self.show_is_soft_deleted
+                } else {
+                    filters = { phase: self.selected_phase ? self.selected_phase.id : undefined }
+                }
             }
-            filters = filters || { phase: self.selected_phase.id }
+
+            filters.page = self.page
+            filters.page_size = self.page_size
+
             CODALAB.api.get_submissions(filters)
-                .done(function (submissions) {
-                    // TODO: should be able to do this with a serializer?
+                .done(function (response) {
+                    let data = response
+                    let results = response
+                    if (response && typeof response === 'object' && response.hasOwnProperty('results')) {
+                        results = response.results || []
+                        self.next = response.next || null
+                        self.previous = response.previous || null
+                        self.total_count = response.count || 0
+                        // prefer page_size from server if present
+                        self.page_size = response.page_size || self.page_size
+                        self.total_pages = Math.max(1, Math.ceil(self.total_count / self.page_size))
+                    } else {
+                        results = response || []
+                        self.next = null
+                        self.previous = null
+                        self.total_count = results.length
+                        self.total_pages = Math.max(1, Math.ceil(self.total_count / self.page_size))
+                    }
+
                     if (opts.admin) {
-                        self.submissions = submissions.map((item) => {
+                        self.submissions = results.map((item) => {
                             item.phase = opts.competition.phases.filter((phase) => {
                                 return phase.id === item.phase
                             })[0]
                             return item
                         })
                     } else {
-                        self.submissions = _.filter(submissions, sub => sub.owner === CODALAB.state.user.username)
+                        self.submissions = _.filter(results, sub => sub.owner === CODALAB.state.user.username)
                     }
+
                     if (!opts.admin) {
                         CODALAB.events.trigger('submissions_loaded', self.submissions)
                     }
+
                     self.csv_link = CODALAB.api.get_submission_csv_URL(filters)
+
                     self.update()
                     self.submission_checked()
 
-                    // Timeout here so loader doesn't flicker
                     _.delay(() => {
                         self.loading = false
                         self.update()
@@ -310,6 +374,34 @@
                 })
         }
 
+
+        self.go_to_page = function (p) {
+            let newPage = parseInt(p, 10)
+            if (isNaN(newPage) || newPage < 1) newPage = 1
+            if (self.total_pages && newPage > self.total_pages) newPage = self.total_pages
+            if (newPage === self.page) return
+            self.page = newPage
+            self.update_submissions()
+        }
+
+        self.change_page_size = function (e) {
+            const val = (e && e.target && e.target.value) ? parseInt(e.target.value, 10) : parseInt(self.page_size, 10)
+            if (!val || val <= 0) return
+            self.page_size = val
+            self.page = 1  // reset to first page when page size changes
+            self.update_submissions()
+        }
+
+        self.handle_page_enter = function (ev) {
+            if (ev.key === 'Enter') {
+                // value du champ input
+                let v = ev.target.value
+                let requested = parseInt(v, 10)
+                if (isNaN(requested)) return
+                self.go_to_page(requested)
+            }
+        }
+
         self.add_to_leaderboard = function (submission) {
             CODALAB.api.add_submission_to_leaderboard(submission.id)
                 .done(function (data) {
@@ -365,6 +457,7 @@
                         filters['phase__competition'] = opts.competition.id
                     }
                 }
+                self.page = 1
                 self.update_submissions(filters)
             }, 100)
         }

From 0733f57f61a73b57d650f2841477c349a3bad0e3 Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Mon, 9 Mar 2026 17:23:37 +0100
Subject: [PATCH 3/6] UI for submisiions pagniation OK, feature complete

---
 src/static/riot/competitions/detail/submission_manager.tag | 3 ---
 1 file changed, 3 deletions(-)

diff --git a/src/static/riot/competitions/detail/submission_manager.tag b/src/static/riot/competitions/detail/submission_manager.tag
index 3eeeb034f..5688f1d44 100644
--- a/src/static/riot/competitions/detail/submission_manager.tag
+++ b/src/static/riot/competitions/detail/submission_manager.tag
@@ -207,10 +207,7 @@
         <div style="display:flex; align-items:center; gap:8px;">
             <label>Per page</label>
             <select class="ui dropdown" value="{ page_size }" onchange="{ change_page_size.bind(this) }">
-                <option value="5">5</option>
-                <option value="10">10</option>
                 <option value="20">20</option>
-                <option value="50">50</option>
             </select>
 
             <div style="margin-left:10px; color: #8c8c8c;">

From 5f98a8a2983dfca860638cbea32b55d5942fcebb Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Tue, 10 Mar 2026 10:15:31 +0100
Subject: [PATCH 4/6] fix some ui issues

---
 .../riot/competitions/detail/submission_manager.tag       | 8 ++------
 1 file changed, 2 insertions(+), 6 deletions(-)

diff --git a/src/static/riot/competitions/detail/submission_manager.tag b/src/static/riot/competitions/detail/submission_manager.tag
index 5688f1d44..70be5511b 100644
--- a/src/static/riot/competitions/detail/submission_manager.tag
+++ b/src/static/riot/competitions/detail/submission_manager.tag
@@ -204,13 +204,9 @@
             </button>
         </div>
 
-        <div style="display:flex; align-items:center; gap:8px;">
-            <label>Per page</label>
-            <select class="ui dropdown" value="{ page_size }" onchange="{ change_page_size.bind(this) }">
-                <option value="20">20</option>
-            </select>
+        <div style="display:flex; align-items:center; gap:8px; margin-left: auto;">
 
-            <div style="margin-left:10px; color: #8c8c8c;">
+            <div style="margin-right: 10px; color: #8c8c8c;">
                 <small>{ total_count || 0 } total</small>
             </div>
     </div>

From fa52922c6f45982975b83f913230b6820345aee7 Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Tue, 10 Mar 2026 11:00:12 +0100
Subject: [PATCH 5/6] fix some ui issues

---
 src/static/riot/competitions/detail/submission_manager.tag | 1 -
 1 file changed, 1 deletion(-)

diff --git a/src/static/riot/competitions/detail/submission_manager.tag b/src/static/riot/competitions/detail/submission_manager.tag
index 70be5511b..b651a8f70 100644
--- a/src/static/riot/competitions/detail/submission_manager.tag
+++ b/src/static/riot/competitions/detail/submission_manager.tag
@@ -227,7 +227,6 @@
 
                     <div if="{is_admin()}" data-tab="admin" class="parent-modal item">Admin</div>
 
-                    <!-- Sometimes submissions end up in a bad state with no children..  -->
                     <div class="item" if="{_.get(selected_submission, 'children').length === 0}">
                         <i style="padding: 5px;">ERROR: Submission is a parent, but has no children. There was an error
                             during creation.</i>

From 50c3809df517b71b6566717dedefeecc345365e9 Mon Sep 17 00:00:00 2001
From: Idir Chikhoune <idir.chikhoune@lisn.fr>
Date: Wed, 11 Mar 2026 12:40:31 +0100
Subject: [PATCH 6/6] dynamic pagination class + UI for submissions

---
 src/apps/api/pagination.py                    | 61 +++++++++++++++++++
 src/apps/api/views/submissions.py             |  4 +-
 .../detail/submission_manager.tag             | 47 +++++++++++---
 3 files changed, 102 insertions(+), 10 deletions(-)

diff --git a/src/apps/api/pagination.py b/src/apps/api/pagination.py
index 5f3cbdc41..f0e07d5a0 100644
--- a/src/apps/api/pagination.py
+++ b/src/apps/api/pagination.py
@@ -22,3 +22,64 @@ def get_paginated_response(self, data):
             'page_size': self.page_size,
             'results': data
         })
+
+
+class DynamicChoicePagination(PageNumberPagination):
+    """
+    Pagination dynamique pour l'UI :
+    - défaut : 50
+    - valeurs autorisées côté client : 50, 100, 500, all
+    - si page_size=all => on renvoie tous les objets (dans la limite de max_page_size)
+    """
+    page_size = 50
+    page_size_query_param = 'page_size'
+    max_page_size = 1000
+    _allowed_sizes = (50, 100, 500, 'all')
+
+    def get_page_size(self, request):
+        raw = request.query_params.get(self.page_size_query_param)
+        if raw is None:
+            return self.page_size
+
+        raw_lower = str(raw).lower()
+        if raw_lower == 'all':
+            return None
+
+        try:
+            val = int(raw)
+        except (TypeError, ValueError):
+            return self.page_size
+
+        if val in (50, 100, 500):
+            return val
+        return self.page_size
+
+    def paginate_queryset(self, queryset, request, view=None):
+        raw = request.query_params.get(self.page_size_query_param)
+        self.requested_page_size = raw if raw is not None else str(self.page_size)
+
+        if raw is not None and str(raw).lower() == 'all':
+            try:
+                total = queryset.count()
+            except Exception:
+                total = self.max_page_size
+            self.page_size = total if (isinstance(total, int) and total > 0) else 1
+        else:
+            page_size = self.get_page_size(request)
+            if page_size is None:
+                page_size = self.page_size
+            self.page_size = page_size
+
+        return super().paginate_queryset(queryset, request, view)
+
+    def get_paginated_response(self, data):
+        page_size_value = self.requested_page_size if getattr(self, 'requested_page_size', None) is not None else self.page_size
+
+        return Response({
+            'next': self.get_next_link(),
+            'previous': self.get_previous_link(),
+            'count': self.page.paginator.count,
+            'page_size': page_size_value,
+            'results': data,
+            'allowed_page_sizes': [50, 100, 500, 'all'],
+        })
\ No newline at end of file
diff --git a/src/apps/api/views/submissions.py b/src/apps/api/views/submissions.py
index 2d95e2e06..89b77a185 100644
--- a/src/apps/api/views/submissions.py
+++ b/src/apps/api/views/submissions.py
@@ -16,7 +16,7 @@
 from django.core.files.base import ContentFile
 
 from profiles.models import Organization, Membership
-from api.pagination import LargePagination
+from api.pagination import DynamicChoicePagination
 from tasks.models import Task
 from api.serializers.submissions import SubmissionCreationSerializer, SubmissionSerializer, SubmissionFilesSerializer, SubmissionDetailSerializer
 from competitions.models import Submission, SubmissionDetails, Phase, CompetitionParticipant
@@ -33,7 +33,7 @@ class SubmissionViewSet(ModelViewSet):
     filter_fields = ('phase__competition', 'phase', 'status', 'is_soft_deleted')
     search_fields = ('data__data_file', 'description', 'name', 'owner__username')
     renderer_classes = api_settings.DEFAULT_RENDERER_CLASSES + [renderers.CSVRenderer]
-    pagination_class = LargePagination
+    pagination_class = DynamicChoicePagination
 
     def check_object_permissions(self, request, obj):
         if self.action in ['submission_leaderboard_connection']:
diff --git a/src/static/riot/competitions/detail/submission_manager.tag b/src/static/riot/competitions/detail/submission_manager.tag
index b651a8f70..a0dc6a90a 100644
--- a/src/static/riot/competitions/detail/submission_manager.tag
+++ b/src/static/riot/competitions/detail/submission_manager.tag
@@ -204,7 +204,14 @@
             </button>
         </div>
 
-        <div style="display:flex; align-items:center; gap:8px; margin-left: auto;">
+        <div style="display:flex; align-items:center; gap:8px;">
+            <label>Per page</label>
+            <select class="ui dropdown" value="{ page_size }" onchange="{ change_page_size.bind(this) }">
+                <option value="50">50</option>
+                <option value="100">100</option>
+                <option value="500">500</option>
+                <option value="all">all</option>
+            </select>
 
             <div style="margin-right: 10px; color: #8c8c8c;">
                 <small>{ total_count || 0 } total</small>
@@ -239,6 +246,8 @@
                         show_visualization="{opts.competition.enable_detailed_results}"
                         submission="{child}"></submission-modal>
                 </div>
+
+
                 <div class="ui tab" style="height: 565px; overflow: auto;" data-tab="admin" if="{is_admin()}">
                     <submission-scores leaderboards="{leaderboards}"></submission-scores>
                 </div>
@@ -258,7 +267,7 @@
         self.show_is_soft_deleted = false
 
         self.page = 1
-        self.page_size = 20
+        self.page_size = 50
         self.total_count = 0
         self.total_pages = 1
         self.next = null
@@ -325,9 +334,22 @@
                         self.next = response.next || null
                         self.previous = response.previous || null
                         self.total_count = response.count || 0
-                        // prefer page_size from server if present
-                        self.page_size = response.page_size || self.page_size
-                        self.total_pages = Math.max(1, Math.ceil(self.total_count / self.page_size))
+                        if (response && typeof response.page_size !== 'undefined') {
+                            const rsp = String(response.page_size).toLowerCase()
+                            if (rsp === 'all') {
+                                self.page_size = 'all'
+                            } else {
+                                const n = Number(response.page_size)
+                                self.page_size = isNaN(n) ? self.page_size : n
+                            }
+                        }
+
+                        if (String(self.page_size).toLowerCase() === 'all') {
+                            self.total_pages = 1
+                        } else {
+                            const ps = Number(self.page_size) || 1
+                            self.total_pages = Math.max(1, Math.ceil(self.total_count / ps))
+                        }
                     } else {
                         results = response || []
                         self.next = null
@@ -377,9 +399,18 @@
         }
 
         self.change_page_size = function (e) {
-            const val = (e && e.target && e.target.value) ? parseInt(e.target.value, 10) : parseInt(self.page_size, 10)
-            if (!val || val <= 0) return
-            self.page_size = val
+            const raw = (e && e.target && typeof e.target.value !== 'undefined') ? String(e.target.value).toLowerCase() : String(self.page_size).toLowerCase()
+
+            if (raw === 'all') {
+                self.page_size = 'all'
+            } else {
+                const val = parseInt(raw, 10)
+                if (isNaN(val) || val <= 0) return
+                // n'autorise que 50,100,500
+                if (![50, 100, 500].includes(val)) return
+                self.page_size = val
+            }
+
             self.page = 1  // reset to first page when page size changes
             self.update_submissions()
         }