Skip to content

Conversation

@haroun
Copy link
Contributor

@haroun haroun commented Nov 13, 2025

Summary

Add failure notifications to batch operation

What are the specific steps to test this change?

You need apostrophecms/apostrophe#5157 to test this.

  1. Use a batch operation that that fails totally or only for some of the run.
  2. Check the frontend notification

What kind of change does this PR introduce?

(Check at least one)

  • Bug fix
  • New feature
  • Refactor
  • Documentation
  • Build-related changes
  • Other

Make sure the PR fulfills these requirements:

  • It includes a) the existing issue ID being resolved, b) a convincing reason for adding this feature, or c) a clear description of the bug it resolves
  • The changelog is updated
  • Related documentation has been updated
  • Related tests have been updated

If adding a new feature without an already open issue, it's best to open a feature request issue first and wait for approval before working on it.

Other information:

@haroun haroun self-assigned this Nov 13, 2025
@linear
Copy link

linear bot commented Nov 13, 2025

@haroun haroun marked this pull request as ready for review November 14, 2025 09:51
@haroun haroun requested a review from ValJed November 14, 2025 09:52
@haroun haroun merged commit 943bfd6 into main Nov 24, 2025
12 checks passed
@haroun haroun deleted the pro-8172-notify-batch-failures branch November 24, 2025 14:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants