darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2025-12-15 09:42:38 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #677 from alphagov/no-jobs-on-api-get

Browse Source 
No jobs on api get
This commit is contained in:
minglis
2016-09-19 12:31:04 +01:00
committed by GitHub
parent aeba205b31 d4971cc45f
commit 1d28f09c22
9 changed files with 431 additions and 34 deletions
Download Patch File Download Diff File Expand all files Collapse all files

4
app/notifications/rest.py
View File

@@ -168,6 +168,7 @@ def get_notification_by_id(notification_id):
@notifications.route('/notifications', methods=['GET'])
def get_all_notifications():
data = notifications_filter_schema.load(request.args).data
include_jobs = data.get('include_jobs', False)
page = data['page'] if 'page' in data else 1
page_size = data['page_size'] if 'page_size' in data else current_app.config.get('PAGE_SIZE')
limit_days = data.get('limit_days')
@@ -179,7 +180,8 @@ def get_all_notifications():
page=page,
page_size=page_size,
limit_days=limit_days,
key_type=api_user.key_type)
key_type=api_user.key_type,
include_jobs=include_jobs)
return jsonify(
notifications=notification_with_personalisation_schema.dump(pagination.items, many=True).data,
page_size=page_size,