Skip to content

Commit

Permalink
Merge pull request #1288 from GSA/notify-api-1287
Browse files Browse the repository at this point in the history
change data retention to 7 days, 23 hours
  • Loading branch information
heyitsmebev authored Aug 22, 2024
2 parents 9b70fb9 + 030a9b5 commit 0e33297
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 1 deletion.
3 changes: 2 additions & 1 deletion app/dao/jobs_dao.py
Original file line number Diff line number Diff line change
Expand Up @@ -188,7 +188,8 @@ def dao_get_jobs_older_than_data_retention(notification_types):
.all()
)

end_date = today - timedelta(days=7)
# notify-api-1287, make default data retention 7 days, 23 hours
end_date = today - timedelta(days=7, hours=23)
for notification_type in notification_types:
services_with_data_retention = [
x.service_id
Expand Down
9 changes: 9 additions & 0 deletions docs/all.md
Original file line number Diff line number Diff line change
Expand Up @@ -1425,3 +1425,12 @@ For application 'notify-api-sandbox': Service instance 'notify-api-rds-sandbox'
```
Run `cf target -o gsa-tts-benefits-studio -s notify-sandbox` before pushing to the Sandbox
### Searchable tags for 'hot' items
Note: better to search on space 'notify-production' rather than specifically for admin or api
#notify-admin-1200 (job cache regeneration)
#notify-admin-1505 (general login issues)
#notify-admin-1701 (wrong sender phone number)
#notify-admin-1859 (job is created with created_at being the wrong time)

0 comments on commit 0e33297

Please sign in to comment.