darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-30 14:31:57 -05:00
Code Issues Packages Projects Releases Wiki Activity

Document area that may be doing pagination links when not needed

Browse Source
This commit is contained in:
David McDonald
2021-12-03 17:30:12 +00:00
parent c68d1a2f23
commit e8dd136678
1 changed files with 3 additions and 0 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/service/rest.py
View File

@@ -542,6 +542,9 @@ def search_for_notification_by_to_field(service_id, search_term, statuses, notif
)
return jsonify(
notifications=notification_with_template_schema.dump(results.items, many=True).data,
# TODO: this may be a bug to include the pagination links as currently `search_for_notification_by_to_field`
# hardcodes the pages of results to always be the first page so not sure what benefit is to show a link to
# page 2 which would have the page parameter ignored
links=pagination_links(
results,
'.get_all_notifications_for_service',