Skip to content

Update default background job worker options configuration in docs#25292

Merged
EngincanV merged 1 commit intorel-10.2from
fix/25291-default-background-job-worker-options-configure
Apr 20, 2026
Merged

Update default background job worker options configuration in docs#25292
EngincanV merged 1 commit intorel-10.2from
fix/25291-default-background-job-worker-options-configure

Conversation

@maliming
Copy link
Copy Markdown
Member

Fix #25291

PreConfigure<AbpBackgroundJobWorkerOptions> actions are never applied because neither AbpBackgroundJobsModule nor AbpBackgroundJobsAbstractionsModule calls ExecutePreConfiguredActions<AbpBackgroundJobWorkerOptions>(). Switch the docs to ConfigureServices + Configure<AbpBackgroundJobWorkerOptions> to match the Hangfire/RabbitMQ samples and the fix in #23977.

Copilot AI review requested due to automatic review settings April 18, 2026 10:41
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates ABP framework documentation to use the correct options-configuration pattern for AbpBackgroundJobWorkerOptions.ApplicationName, ensuring the sample actually takes effect for Default background jobs/workers when multiple apps share the same storage (fixing the misguidance described in #25291 / aligning with #23977 context).

Changes:

  • Switches the sample override from PreConfigureServices to ConfigureServices.
  • Switches the sample from PreConfigure<AbpBackgroundJobWorkerOptions> to Configure<AbpBackgroundJobWorkerOptions> in both Background Workers and Background Jobs docs.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
docs/en/framework/infrastructure/background-workers/index.md Updates Default background job/worker isolation sample to use ConfigureServices + Configure<AbpBackgroundJobWorkerOptions>.
docs/en/framework/infrastructure/background-jobs/index.md Same doc fix for the Background Jobs page to ensure ApplicationName configuration is applied.

@maliming maliming requested a review from EngincanV April 20, 2026 01:43
@EngincanV EngincanV merged commit c6d75a0 into rel-10.2 Apr 20, 2026
6 checks passed
@EngincanV EngincanV deleted the fix/25291-default-background-job-worker-options-configure branch April 20, 2026 11:58
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