Skip to content

[stable34] feat: adding an option to exclude users from the acitivity log#2808

Open
backportbot[bot] wants to merge 1 commit into
stable34from
backport/2626/stable34
Open

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

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 #4060

Run Properties:  status check failed Failed #4060  •  git commit a76bf258a9: [stable34] feat: adding an option to exclude users from the acitivity log
Project Activity
Branch Review backport/2626/stable34
Run status status check failed Failed #4060
Run duration 02m 54s
Commit git commit a76bf258a9: [stable34] 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 1
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 8
View all changes introduced in this branch ↗︎

Tests for review

Failed  cypress/e2e/settings.cy.ts • 1 failed test • Run E2E

View Output

Test Artifacts
Check that user's settings survive a reload > Form survive a reload Test Replay Screenshots

@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