darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-08-19 14:08:47 -04:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #184 from alphagov/notifications-return-change

Browse Source 
Align notifications api with docs and other endpoints
This commit is contained in:
Adam Shimali
2016-03-24 16:24:43 +00:00
parent c7622edba8 7bc81528bb
commit ed1d988bf4
2 changed files with 8 additions and 6 deletions
Download Patch File Download Diff File Expand all files Collapse all files

5
app/notifications/rest.py
View File

@@ -215,7 +215,7 @@ def process_firetext_response():
@notifications.route('/notifications/<uuid:notification_id>', methods=['GET'])
def get_notifications(notification_id):
notification = notifications_dao.get_notification(api_user['client'], notification_id)
return jsonify({'notification': notification_status_schema.dump(notification).data}), 200
return jsonify(data={"notification": notification_status_schema.dump(notification).data}), 200
@notifications.route('/notifications', methods=['GET'])
@@ -380,4 +380,5 @@ def send_notification(notification_type):
encryption.encrypt(notification),
datetime.utcnow().strftime(DATETIME_FORMAT)
), queue='email')
return jsonify({'notification_id': notification_id}), 201
return jsonify(data={"notification": {"id": notification_id}}), 201