entrouvert
/
authentic
16
0
Fork 0
Code Issues Pull Requests 34 Releases Activity

api: make users pagination offset as a setting (#81100) #124

Merged
smihai merged 1 commits from wip/81100-api-make-users-pagination-cutoff-as-setting into main 2023-09-20 11:41:30 +02:00
Conversation 0 Commits 1 Files Changed 3 +24
smihai commented 2023-09-12 11:59:49 +02:00
Owner
Copy Link
No description provided.
tnoel requested changes 2023-09-12 12:14:07 +02:00
src/authentic2/settings.py Outdated
@ -398,6 +398,8 @@ PHONE_COUNTRY_CODES = {
DEFAULT_COUNTRY_CODE = '33'
REST_FRAMEWORK_CURSOR_OFFSET_CUTOFF = 1000
tnoel commented 2023-09-12 12:14:03 +02:00
Owner
Copy Link

Ca ressemble à un paramètre générique alors qu'il ne va s'appliquer qu'à /api/users, je serais pour un nom qui parle plus, genre « limite du nombres d'utilisateurs à renvoyer par l'API ».

Ca ressemble à un paramètre générique alors qu'il ne va s'appliquer qu'à /api/users, je serais pour un nom qui parle plus, genre « limite du nombres d'utilisateurs à renvoyer par l'API ».
smihai force-pushed wip/81100-api-make-users-pagination-cutoff-as-setting from 4b6a810a7c to ca4889dd51 2023-09-12 16:26:52 +02:00 Compare
pmarillonnet reviewed 2023-09-13 17:18:16 +02:00
src/authentic2/api_views.py
@ -716,6 +716,7 @@ class FreeTextSearchFilter(BaseFilterBackend):
class UsersAPIPagination(pagination.CursorPagination):
page_size_query_param = 'limit'
max_page_size = 100
pmarillonnet commented 2023-09-13 17:18:16 +02:00
Owner
Copy Link

Quelle différence avec ce paramètre déjà existant ? Est-ce simplement qu’il a changé de nom ?

Quelle différence avec ce paramètre déjà existant ? Est-ce simplement qu’il a changé de nom ?
smihai commented 2023-09-13 17:21:58 +02:00
Author
Owner
Copy Link

max_page_size définit la taille maximale des résultats.
Ce nouveau paramètre limite à l'API à retourner max 1000 résultats.

L'idée est de pouvoir la modifier lorsque le résultat du filter dépasse 1000 résultats.

`max_page_size` définit la taille maximale des résultats. Ce nouveau paramètre limite à l'API à retourner max 1000 résultats. L'idée est de pouvoir la modifier lorsque le résultat du filter dépasse 1000 résultats.
pmarillonnet commented 2023-09-13 17:30:51 +02:00
Owner
Copy Link

Ah oui ok pigé, merci.

Ah oui ok pigé, merci.
smihai force-pushed wip/81100-api-make-users-pagination-cutoff-as-setting from ca4889dd51 to 56b127ca03 2023-09-20 11:03:27 +02:00 Compare
smihai force-pushed wip/81100-api-make-users-pagination-cutoff-as-setting from 56b127ca03 to f72ad68538 2023-09-20 11:19:49 +02:00 Compare
smihai changed title from WIP: api: make users pagination offset as a setting (#81100) to api: make users pagination offset as a setting (#81100) 2023-09-20 11:32:43 +02:00
pmarillonnet approved these changes 2023-09-20 11:39:47 +02:00
smihai merged commit 432f737c55 into main 2023-09-20 11:41:30 +02:00
smihai deleted branch wip/81100-api-make-users-pagination-cutoff-as-setting 2023-09-20 11:41:30 +02:00
Sign in to join this conversation.
Reviewers
No reviewers
tnoel
pmarillonnet
Labels
Clear labels
No items
No Label
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
3 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: entrouvert/authentic#124
No description provided.
Delete Branch "wip/81100-api-make-users-pagination-cutoff-as-setting"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?