Skip to content

Draft: Rename "audit log" css classes to "audit event" class names

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

What does this MR do and why?

Rename "audit log" css classes to "audit event" class names

This is another commit related to issue 292667

In this commit, I am renaming css classes and js data hooks.

As css class names and data attributes are used by JS, specs, components, etc., this is a moderatley risky 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
  2. TBD

MR acceptance checklist

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

Merge request reports