Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[stable30] fix: do not send daily digest email to user who is disabled #1829

Merged
merged 6 commits into from
Oct 29, 2024

Conversation

backportbot[bot]
Copy link

@backportbot backportbot bot commented Oct 28, 2024

Backport of #1803

Warning, This backport's changes differ from the original and might be incomplete ⚠️

Todo

  • Review and resolve any conflicts
  • Amend HEAD commit to remove the line stating to skip CI

Learn more about backports at https://docs.nextcloud.com/server/stable/go.php?to=developer-backports.

Signed-off-by: yemkareems <[email protected]>
Copy link

cypress bot commented Oct 29, 2024

Activity    Run #2064

Run Properties:  status check failed Failed #2064  •  git commit 3002250757: [stable30] fix: do not send daily digest email to user who is disabled
Project Activity
Branch Review backport/1803/stable30
Run status status check failed Failed #2064
Run duration 03m 19s
Commit git commit 3002250757: [stable30] fix: do not send daily digest email to user who is disabled
Committer backportbot[bot]
View all properties for this run ↗︎

Test results
Tests that failed  Failures 2
Tests that were flaky  Flaky 1
Tests that did not run due to a developer annotating a test with .skip  Pending 0
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/sidebar.cy.ts • 2 failed tests • Run E2E

View Output

Test Artifacts
Check activity listing in the sidebar > Has share activity Test Replay Screenshots
Check activity listing in the sidebar > Has tag activity Test Replay Screenshots
Flakiness  cypress/e2e/sidebar.cy.ts • 1 flaky test • Run E2E

View Output

Test Artifacts
Check activity listing in the sidebar > Has comment activity Test Replay Screenshots

@yemkareems yemkareems marked this pull request as ready for review October 29, 2024 08:38
@artonge artonge merged commit dd72d87 into stable30 Oct 29, 2024
49 of 51 checks passed
@artonge artonge deleted the backport/1803/stable30 branch October 29, 2024 08:54
@Altahrim Altahrim mentioned this pull request Oct 30, 2024
2 tasks
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