Fix unnecessary “next” link when API returns fewer notification requests than requested (#31541)

This commit is contained in:
Claire 2024-08-22 15:42:04 +02:00 committed by GitHub
parent abe292b0ec
commit 98237207e6
No known key found for this signature in database
GPG key ID: B5690EEEBB952194

View file

@ -75,13 +75,17 @@ class Api::V1::Notifications::RequestsController < Api::BaseController
end end
def next_path def next_path
api_v1_notifications_requests_url pagination_params(max_id: pagination_max_id) unless @requests.empty? api_v1_notifications_requests_url pagination_params(max_id: pagination_max_id) if records_continue?
end end
def prev_path def prev_path
api_v1_notifications_requests_url pagination_params(min_id: pagination_since_id) unless @requests.empty? api_v1_notifications_requests_url pagination_params(min_id: pagination_since_id) unless @requests.empty?
end end
def records_continue?
@requests.size == limit_param(DEFAULT_ACCOUNTS_LIMIT)
end
def pagination_max_id def pagination_max_id
@requests.last.id @requests.last.id
end end