Skip to content

Conversation

@tishmen
Copy link

@tishmen tishmen commented Oct 1, 2025

Scope

  • queue_job_cron, queue_job_cron_jobrunner

Depends on

Summary

  • Migrate to Odoo 19; tests included. No functional changes beyond compatibility.

Temporary references

  • test-requirements.txt adds temporary dependencies to the base PR for CI/runboat.
    This will be removed once the base PR merges.

Pre-commit

  • Ran locally; auto-fixes committed.

Tests

  • CI will run the suite using the referenced base PR. Local isolated runs may skip pieces if queue_job is absent.

@tishmen tishmen changed the title [19.0] queue_job_cron: migrate + tests [19.0] [MIG] queue_job_cron: migrate Oct 1, 2025
@sbidoul sbidoul added this to the 19.0 milestone Jan 2, 2026
@tishmen tishmen force-pushed the pr4/queue_job_cron branch from 0b87929 to 4e12088 Compare January 9, 2026 16:01
@tishmen
Copy link
Author

tishmen commented Jan 9, 2026

Hi @sbidoul, I rebased this PR on the latest 19.0, removed the temporary test-requirements.txt pointing to refs/pull/840/head, restored requirements.txt, and squashed everything into a single commit. CI is green on the new head.

@sbidoul
Copy link
Member

sbidoul commented Jan 9, 2026

Please do different PRs for queue_job_cron and queue_job_cron_jobrunner. Despite the similar name they are very different things.

Split from OCA#843 as requested; this commit only migrates queue_job_cron.
@tishmen tishmen force-pushed the pr4/queue_job_cron branch from 4e12088 to 092f849 Compare January 9, 2026 16:52
tishmen added a commit to tishmen/queue that referenced this pull request Jan 9, 2026
Split from OCA#843 as requested; this commit only migrates queue_job_cron_jobrunner.
@tishmen
Copy link
Author

tishmen commented Jan 9, 2026

Done: split as requested. This PR (#843) now contains only queue_job_cron. The queue_job_cron_jobrunner migration is in #876.

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.

2 participants