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

Add more logging for getting next page logic

Browse Source
This commit is contained in:
Imdad Ahad
2017-03-15 10:29:38 +00:00
parent f5166bec79
commit fc8bbc5d6f
1 changed files with 7 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

8
app/utils.py
View File

@@ -154,9 +154,9 @@ def generate_notifications_csv(**kwargs):
kwargs['page'] = 1 kwargs['page'] = 1
while kwargs['page']: while kwargs['page']:
current_app.logger.info('Current kwargs: {}'.format(kwargs))
notifications_resp = notification_api_client.get_notifications_for_service(**kwargs) notifications_resp = notification_api_client.get_notifications_for_service(**kwargs)
current_app.logger.info('Total notifications for csv job: {}'.format(notifications_resp['total'])) current_app.logger.info('Total notifications for csv job: {}'.format(notifications_resp['total']))
current_app.logger.info('Notification response links: {}'.format(notifications_resp['links']))
notifications_list = notifications_resp['notifications'] notifications_list = notifications_resp['notifications']
for x in notifications_list: for x in notifications_list:
csvwriter.writerow(format_notification_for_csv(x)) csvwriter.writerow(format_notification_for_csv(x))
@@ -166,8 +166,14 @@ def generate_notifications_csv(**kwargs):
yield line yield line
if notifications_resp['links'].get('next'): if notifications_resp['links'].get('next'):
current_app.logger.info('Next link exists')
current_app.logger.info('Notification response links: {}'.format(notifications_resp['links']))
current_app.logger.info('Current page {}'.format(kwargs['page']))
kwargs['page'] += 1 kwargs['page'] += 1
else: else:
current_app.logger.info('No next link exists')
current_app.logger.info('Notification response links: {}'.format(notifications_resp['links']))
current_app.logger.info('Current page {}'.format(kwargs['page']))
return return