Skip to content

[18.0][FIX] queue_job: remove create/delete permissions - #927

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
guewen:18.0-remove-manager-cd-permission
May 1, 2026
Merged

[18.0][FIX] queue_job: remove create/delete permissions#927
OCA-git-bot merged 1 commit into
OCA:18.0from
guewen:18.0-remove-manager-cd-permission

Conversation

@guewen

Copy link
Copy Markdown
Member

Jobs are always created through sudo() in the Job class, and always deleted by the cron, never by users.

See discussion on #802 (comment)

Jobs are always created through sudo() in the Job class, and always
deleted by the cron, never by users.
See discussion on OCA#802 (comment)
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@guewenguewen changed the title [FIX] queue_job: remove create/delete permissions[18.0][FIX] queue_job: remove create/delete permissionsApr 30, 2026

@amh-mwamh-mw left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Functionally tested locally. LGTM.

@guewen

Copy link
Copy Markdown
MemberAuthor

/ocabot merge patch

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-927-by-guewen-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 0e74bfe into OCA:18.0May 1, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

Sign up for freeto 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

@guewen@OCA-git-bot@amh-mw@sbidoul