darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-02-01 15:46:07 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #812 from alphagov/plat-adm-dates

Browse Source 
make sure we convert strings to dates before we use them
This commit is contained in:
Leo Hemsted
2017-01-31 16:24:30 +00:00
committed by GitHub
parent d0b467b2fb cd47dbd36b
commit 855a412c90
2 changed files with 40 additions and 3 deletions
Download Patch File Download Diff File Expand all files Collapse all files

7
app/service/rest.py
View File

@@ -68,9 +68,12 @@ def get_services():
detailed = request.args.get('detailed') == 'True'
user_id = request.args.get('user_id', None)
include_from_test_key = request.args.get('include_from_test_key', 'True') != 'False'
# If start and end date are not set, we are expecting today's stats.
start_date = request.args.get('start_date', datetime.utcnow().date())
end_date = request.args.get('end_date', datetime.utcnow().date())
today = str(datetime.utcnow().date())
start_date = datetime.strptime(request.args.get('start_date', today), '%Y-%m-%d').date()
end_date = datetime.strptime(request.args.get('end_date', today), '%Y-%m-%d').date()
if user_id:
services = dao_fetch_all_services_by_user(user_id, only_active)