Skip to content

Conversation

@ivantodorovich
Copy link
Contributor

We should be able to cancel jobs waiting dependencies

@OCA-git-bot
Copy link
Contributor

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

@florentx
Copy link
Contributor

florentx commented Jun 10, 2025

I assume that you need to change it here as well?
https://github.com/OCA/queue/blob/17.0/queue_job/views/queue_job_views.xml#L28

Copy link
Contributor

@SilvioC2C SilvioC2C left a comment

Choose a reason for hiding this comment

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

LGTM, imho @florentx is right about updating the view too

@SilvioC2C SilvioC2C force-pushed the 17.0-queue-fix-cancel branch from d0e1df2 to cf01e1f Compare June 10, 2025 14:57
@ivantodorovich ivantodorovich force-pushed the 17.0-queue-fix-cancel branch from cf01e1f to 2c92b5d Compare June 10, 2025 18:09
@ivantodorovich
Copy link
Contributor Author

Absolutely. It's done

@sbidoul sbidoul added this to the 17.0 milestone Jan 2, 2026
@sbidoul
Copy link
Member

sbidoul commented Jan 2, 2026

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 17.0-ocabot-merge-pr-785-by-sbidoul-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit de0a304 into OCA:17.0 Jan 2, 2026
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

Congratulations, your PR was merged at ada1035. 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.

7 participants