darkhelm/notifications-admin
1
0
Fork 0
mirror of https://github.com/GSA/notifications-admin.git synced 2026-08-26 09:13:58 -04:00
Code Issues Packages Projects Releases Wiki Activity

Ask API for less detailed JSON for all templates

Browse Source 
This should speed things up by:
- less time waiting for big blobs of JSON to come from Redis or the API
- less time spent deserialising big blobs of JSON
This commit is contained in:
Chris Hill-Scott
2020-06-19 10:32:44 +01:00
parent c7543df39b
commit 51e99bd828
2 changed files with 2 additions and 2 deletions
Download Patch File Download Diff File Expand all files Collapse all files

2
app/notify_client/service_api_client.py
View File

@@ -276,7 +276,7 @@ class ServiceAPIClient(NotifyAdminAPIClient):
""" """
Retrieve all templates for service. Retrieve all templates for service.
""" """
endpoint = '/service/{service_id}/template'.format( endpoint = '/service/{service_id}/template?detailed=False'.format(
service_id=service_id) service_id=service_id)
return self.get(endpoint) return self.get(endpoint)

2
tests/app/notify_client/test_service_api_client.py
View File

@@ -275,7 +275,7 @@ def test_client_returns_count_of_service_templates(
], ],
None, None,
[ [
call('/service/{}/template'.format(SERVICE_ONE_ID)) call('/service/{}/template?detailed=False'.format(SERVICE_ONE_ID))
], ],
[ [
call( call(