[13.0][FIX] queue_job_cron: channel_id must be storable.

[LoisRForgeFlow]

Otherwise, you cannot use any channel other than default (
root.ir_cron)

The struggle was real without this fix 😢 :

After the fix life is great 😄 :

@ForgeFlow

[AaronHForgeFlow]

👍 makes sense

[AdriaGForgeFlow]

LGTM 👍

[LoisRForgeFlow]

@guewen Do you think this is enough?, or it is better to use an extra field manual_channel_id and rely on that one for the compute ensuring that it won't get back to False if the compute is triggered.

[guewen]

@guewen Do you think this is enough?, or it is better to use an extra field manual_channel_id and rely on that one for the compute ensuring that it won't get back to False if the compute is triggered.

With the new compute methods of 13.0, we can make it work like a default value:

for cron in self:
if cron.channel_id:
continue
if cron.run_as_queue_job:
cron.channel_id = ...

That being said, I'm not sure why this field has to be computed.

[OCA-git-bot]

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). 🤖

[OCA-git-bot]

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). 🤖

[LoisRForgeFlow]

@guewen I added the continue clause, I think that makes sense. For me this is good to merge.

[guewen]

/ocabot merge patch

[OCA-git-bot]

What a great day to merge this nice PR. Let's do it!
Prepared branch 13.0-ocabot-merge-pr-355-by-guewen-bump-patch, awaiting test results.

[OCA-git-bot]

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