Skip to content

[REF] queue_job: documentation cleanup - #886

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
ursais:18.0-ref-queue-docs
Feb 3, 2026
Merged

[REF] queue_job: documentation cleanup#886
OCA-git-bot merged 1 commit into
OCA:18.0from
ursais:18.0-ref-queue-docs

Conversation

@RLeeGML

@RLeeGMLRLeeGML commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

Cleanup of documentation in python file, is already present in readme file. Also adds clarity on env vars and conf file options

@OCA-git-bot

OCA-git-bot commented Jan 26, 2026

Copy link
Copy Markdown
Contributor

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

@RLeeGMLRLeeGML mentioned this pull request Jan 26, 2026
Closed
@RLeeGML

RLeeGML commented Jan 26, 2026

Copy link
Copy Markdown
ContributorAuthor

@sbidoul First part of the split PR

amh-mw
amh-mw approved these changes Jan 26, 2026
View reviewed changes

@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.

Confirmed that deleted comments appear in CONFIGURE.md

@RLeeGMLRLeeGML mentioned this pull request Jan 26, 2026
Open
@sbidoul

sbidoul commented Feb 2, 2026

Copy link
Copy Markdown
Member

Confirmed that deleted comments appear in CONFIGURE.md

@guewen I think there used to be a sphinx docs generator that was picking up those comments. I suppose we can consider this is lost in time? Or is it something that's worth reviving?

@guewen

guewen commented Feb 3, 2026

Copy link
Copy Markdown
Member

I suppose we can consider this is lost in time?

Yeah I would say so, better put time in the readme.

Thanks for the cleaning @RLeeOSI

@sbidoul

sbidoul commented Feb 3, 2026

Copy link
Copy Markdown
Member

/ocabot merge patch

Thanks!

@OCA-git-bot

OCA-git-bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-886-by-sbidoul-bump-patch, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 9a24a3e into OCA:18.0Feb 3, 2026
7 checks passed
@OCA-git-bot

OCA-git-bot commented Feb 3, 2026

Copy link
Copy Markdown
Contributor

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

This was referenced Mar 23, 2026
Merged
Open
Sign up for freeto join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

@guewenguewenguewen approved these changes
+1 more reviewer
@amh-mwamh-mwamh-mw approved these changes
Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

approvedmerged 🎉Needs FW Port

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

5 participants

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