Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Add a job to delete old attachments #1402

Merged
merged 3 commits into from
Aug 21, 2024
Merged

Conversation

gunndabad
Copy link
Collaborator

@gunndabad gunndabad commented Jul 1, 2024

This adds a nightly job that will delete any attachments from change name/DOB requests and TRN request tasks that have not been modified for 30 days. It also creates a TRS event that contains the attachment's ID, for auditing purposes. A separate PR will follow to sync the event from CRM to TRS.

We're a little light on tests here as we can't override the created/modified timestamps on CRM entities (so we can't create easily entities that appear to be 30 days old).

@gunndabad
Copy link
Collaborator Author

@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 8e4c327 to 797cb28 Compare July 4, 2024 09:57
@gunndabad gunndabad changed the title Add a job to delete old incident attachments Add a job to delete old attachments Jul 4, 2024
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 72fdceb to 25e8303 Compare July 10, 2024 09:19
@gunndabad gunndabad marked this pull request as ready for review July 11, 2024 11:40
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 25e8303 to abaa6bb Compare July 11, 2024 11:40
@gunndabad gunndabad temporarily deployed to pre-production_aks July 11, 2024 12:52 — with GitHub Actions Inactive
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from abaa6bb to 1b8782d Compare August 6, 2024 13:59
@gunndabad gunndabad temporarily deployed to pre-production_aks August 6, 2024 14:03 — with GitHub Actions Inactive
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 1b8782d to 4744c0c Compare August 21, 2024 10:37
@gunndabad gunndabad temporarily deployed to pre-production_aks August 21, 2024 10:41 — with GitHub Actions Inactive
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 19ea633 to 6229781 Compare August 21, 2024 11:42
@gunndabad gunndabad temporarily deployed to pre-production_aks August 21, 2024 11:45 — with GitHub Actions Inactive
@gunndabad gunndabad force-pushed the delete-old-attachments-job branch from 6229781 to aeec6db Compare August 21, 2024 11:48
@gunndabad gunndabad enabled auto-merge (squash) August 21, 2024 11:48
@gunndabad gunndabad merged commit a2317a4 into main Aug 21, 2024
12 of 13 checks passed
@gunndabad gunndabad deleted the delete-old-attachments-job branch August 21, 2024 12:05
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants