darkhelm/notifications-api
1
0
Fork 0
mirror of https://github.com/GSA/notifications-api.git synced 2026-01-04 09:43:33 -05:00
Code Issues Packages Projects Releases Wiki Activity

Merge pull request #1288 from GSA/notify-api-1287

Browse Source 
change data retention to 7 days, 23 hours
This commit is contained in:
Beverly Nguyen
2024-08-22 09:55:09 -07:00
committed by GitHub
parent 9b70fb90d3 030a9b5daa
commit 0e332973ad
2 changed files with 11 additions and 1 deletions
Download Patch File Download Diff File Expand all files Collapse all files

3
app/dao/jobs_dao.py
View File

@@ -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

9
docs/all.md
View File

@@ -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)