Skip to content

[ADD][14.0] queue_job_context#462

Merged
OCA-git-bot merged 1 commit intoOCA:14.0from
AshishHirapara:14.0-queue_job_context
Oct 19, 2022
Merged

[ADD][14.0] queue_job_context#462
OCA-git-bot merged 1 commit intoOCA:14.0from
AshishHirapara:14.0-queue_job_context

Conversation

@AshishHirapara
Copy link
Copy Markdown

initial commit - Add context before queue

Comment thread queue_job_context/__manifest__.py Outdated
Comment thread queue_job_context/readme/CONTRIBUTORS.rst Outdated
Comment thread queue_job_context/readme/CONFIGURE.rst Outdated
@AshishHirapara AshishHirapara force-pushed the 14.0-queue_job_context branch 3 times, most recently from 977f9ab to cd4a6d2 Compare October 14, 2022 03:35
Comment thread queue_job_context/readme/DESCRIPTION.rst Outdated
@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Oct 15, 2022

A small comment about the readme, otherwise looks good, thanks!

@hoangtrann
Copy link
Copy Markdown
Contributor

May I ask why we didn't include this behavior in #433?

@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Oct 18, 2022

May I ask why we didn't include this behavior in #433?

@hoangtrann for backward compatibility reasons.

In 16.0 this must be the default cc/ @baimont

Comment thread queue_job_context/__manifest__.py Outdated
Comment thread queue_job_context/readme/DESCRIPTION.rst Outdated
initial commit - Add context before queue
@sbidoul sbidoul mentioned this pull request Oct 18, 2022
@francesco-ooops
Copy link
Copy Markdown
Contributor

@OCA/connector-infor-maintainers merge?

@sbidoul
Copy link
Copy Markdown
Member

sbidoul commented Oct 19, 2022

/ocabot merge patch

@OCA-git-bot
Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 14.0-ocabot-merge-pr-462-by-sbidoul-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 3616f2f into OCA:14.0 Oct 19, 2022
@OCA-git-bot
Copy link
Copy Markdown
Contributor

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

5 participants