Skip to content

[stable33] feat: adding an option to exclude users from the acitivity log#2809

Open
backportbot[bot] wants to merge 1 commit into
stable33from
backport/2626/stable33
Open

[stable33] feat: adding an option to exclude users from the acitivity log#2809
backportbot[bot] wants to merge 1 commit into
stable33from
backport/2626/stable33

Conversation

@backportbot

@backportbot backportbot Bot commented Jul 21, 2026

Copy link
Copy Markdown

Backport of PR #2626

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

fix: Fixing missing $affectedUser in storeMail()

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

fix: Removing 'all' as an even type filter option for excluded users

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

chore: Adding tests for excluded users

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

fix: Removing dead code from condition

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

fix: logging an error when activity_log_exclude_users looks invalid

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>

fix: Fixing indentation

Signed-off-by: Tamás Bari <adaorcpp@gmail.com>
@cypress

cypress Bot commented Jul 21, 2026

Copy link
Copy Markdown

Activity    Run #4061

Run Properties:  status check passed Passed #4061  •  git commit 4e92517d76: [stable33] feat: adding an option to exclude users from the acitivity log
Project Activity
Branch Review backport/2626/stable33
Run status status check passed Passed #4061
Run duration 02m 13s
Commit git commit 4e92517d76: [stable33] feat: adding an option to exclude users from the acitivity log
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 0
Tests that were flaky  Flaky 0
Tests that did not run due to a developer annotating a test with .skip  Pending 1
Tests that did not run due to a failure in a mocha hook  Skipped 0
Tests that passed  Passing 9
View all changes introduced in this branch ↗︎

@miaulalala miaulalala left a comment

Copy link
Copy Markdown
Collaborator

Choose a reason for hiding this comment

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

Backport of already-reviewed master PR, approving.

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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants