Skip to content

AO3-7307 Created Resque job to clean up old user audits#5791

Open
hunternet93 wants to merge 10 commits into
otwcode:masterfrom
hunternet93:AO3-7307_audits_cleanup_job
Open

AO3-7307 Created Resque job to clean up old user audits#5791
hunternet93 wants to merge 10 commits into
otwcode:masterfrom
hunternet93:AO3-7307_audits_cleanup_job

Conversation

@hunternet93
Copy link
Copy Markdown
Contributor

Issue

https://otwarchive.atlassian.net/browse/AO3-7307

Purpose

This adds a Resque job that will delete audit records older than a configurable limit. A new admin setting is added to exempt particular usernames from deletion.

Credit

Hunter Ada Smith (she/they)

@github-actions github-actions Bot added Has Migrations Contains migrations and therefore needs special attention when deploying Awaiting Review labels May 6, 2026
Copy link
Copy Markdown
Contributor

@pmonfort pmonfort left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code looks good, great work! Left a couple of comments

Comment thread app/jobs/audits_cleanup_job.rb Outdated
Comment thread app/policies/admin_setting_policy.rb
Comment thread app/views/admin/settings/index.html.erb Outdated
@pmonfort
Copy link
Copy Markdown
Contributor

Thanks for addressing my comments, the PR looks good to me. Now we only need Bilka2's approval 👍

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Coder Has Actioned Review Has Migrations Contains migrations and therefore needs special attention when deploying

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants