Skip to content

Rename `audit_logs_params` -> `audit_events_params`

Michael Becker requested to merge feat/292667-pt4 into master

What does this MR do and why?

This is another commit related to issue 292667

In this commit, I renaming some controller concerns and their usages

This should be a fairly low-risk change

Background on the Change

We are deprecating "Audit Log" terminology in favor of "Audit Event" terminology as described in the linked issue

I have used the following regex to identify places in the code where the "Audit Log" terminology is used as opposed to "Audit Event" terminology:

 ag --color --hidden \
    --ignore .git \
    --ignore doc \
    --ignore changelogs \
    --ignore data \
    --ignore locale \
    --ignore "CHANGELOG*" \
    '^.*(?<!features\(admin_|features\(|flags\(|features\()[aA]udit[_-]?\s?[Ll]ogs?'

The first half of the regex ((?<!features\(admin_|features\(|flags\(|features\()) is meant to exclude feature flag names as I will be ignoring those for the overall re-naming effort

How to set up and validate locally

  1. ci passes

MR acceptance checklist

This checklist encourages us to confirm any changes have been analyzed to reduce risks in quality, performance, reliability, security, and maintainability.

Edited by Michael Becker

Merge request reports

Loading