Skip to content

feat(transaction): Track background jobs#707

Draft
ChristophWurst wants to merge 1 commit into
masterfrom
feat/cron/transactions
Draft

feat(transaction): Track background jobs#707
ChristophWurst wants to merge 1 commit into
masterfrom
feat/cron/transactions

Conversation

@ChristophWurst
Copy link
Copy Markdown
Owner

@ChristophWurst ChristophWurst commented Mar 31, 2025

This is a follow-up to #700 and covers background jobs.

Bildschirmfoto vom 2025-03-31 18-36-52

Signed-off-by: Christoph Wurst <christoph@winzerhof-wurst.at>
/** @psalm-suppress TooManyArguments */
$context->registerMiddleware(PerformanceMonitoringMiddleware::class, true);
$context->registerEventListener(AddContentSecurityPolicyEvent::class, CustomCspListener::class);
if (class_exists(BeforeJobExecutedEvent::class)) {
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.

Should not be needed, the class name can be used without existance, just psalm complains

Copy link
Copy Markdown
Owner Author

Choose a reason for hiding this comment

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

I did it for Psalm but it complains in any case

@coderabbitai
Copy link
Copy Markdown

coderabbitai Bot commented Apr 30, 2026

Important

Review skipped

Draft detected.

Please check the settings in the CodeRabbit UI or the .coderabbit.yaml file in this repository. To trigger a single review, invoke the @coderabbitai review command.

⚙️ Run configuration

Configuration used: defaults

Review profile: CHILL

Plan: Pro Plus

Run ID: 3a349599-1644-4702-986b-616609eabe5e

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Comment @coderabbitai help to get the list of available commands and usage tips.

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