Skip to content

[FIX] queue_job_cron_jobrunner: use priority to select job#818

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Solvti:18.0-fix-queue_job_cron_jobrunner-use-priority-to-select-job
Sep 2, 2025
Merged

[FIX] queue_job_cron_jobrunner: use priority to select job#818
OCA-git-bot merged 1 commit into
OCA:18.0from
Solvti:18.0-fix-queue_job_cron_jobrunner-use-priority-to-select-job

Conversation

@StefanWiselka-Solvti
Copy link
Copy Markdown
Contributor

[FIX] queue_job_cron_jobrunner: use priority to select job

  • use FIFO, firt createad job will be treat first
  • if priority are different it take the precedent

Yet we are not using channel priority into account.

Note: FIX has already been applied to branch 17.0.
Ref: f8fcceb

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hi @ivantodorovich,
some modules you are maintaining are being modified, check this out!

Copy link
Copy Markdown

@MarcinSokolowski-Solvti MarcinSokolowski-Solvti left a comment

Choose a reason for hiding this comment

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

LGTM 🚀

@OCA-git-bot
Copy link
Copy Markdown
Contributor

This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖

@ivantodorovich
Copy link
Copy Markdown
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-818-by-ivantodorovich-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit f52ba34 into OCA:18.0 Sep 2, 2025
7 checks passed
@OCA-git-bot
Copy link
Copy Markdown
Contributor

Congratulations, your PR was merged at a6520d9. Thanks a lot for contributing to OCA. ❤️

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.

4 participants