Skip to content

[18.0][ADD] helpdesk_mgmt_mail_autogenerated_header - #1050

Merged
OCA-git-bot merged 1 commit into
OCA:18.0from
Nitrokey:18.0-add-helpdesk_mgmt_mail_autogenerated_header
Sep 8, 2026
Merged

[18.0][ADD] helpdesk_mgmt_mail_autogenerated_header#1050
OCA-git-bot merged 1 commit into
OCA:18.0from
Nitrokey:18.0-add-helpdesk_mgmt_mail_autogenerated_header

Conversation

@jans23

@jans23 jans23 commented Jun 22, 2026

Copy link
Copy Markdown
Contributor

When a helpdesk ticket is created from an auto-generated email (e.g. an
out-of-office reply carrying Auto-Submitted: auto-replied or
X-Auto-Response-Suppress: AutoReply/All), the stage welcome email
configured on the ticket stage must not be sent back to avoid an email loop.

This module integrates helpdesk_mgmt with mail_autogenerated_header:
the latter detects such headers on incoming messages and sets a context key
that this module reads in _track_template_skip_autoreply to suppress the
outgoing welcome email.

If follows a similar approach as #1051.

@OCA-git-bot OCA-git-bot added mod:helpdesk_mgmt Module helpdesk_mgmt series:18.0 mod:helpdesk_mgmt_mail_autogenerated_header Module helpdesk_mgmt_mail_autogenerated_header labels Jun 22, 2026
@jans23
jans23 force-pushed the 18.0-add-helpdesk_mgmt_mail_autogenerated_header branch 2 times, most recently from 9c49520 to cd83274 Compare June 22, 2026 14:30
@jans23
jans23 force-pushed the 18.0-add-helpdesk_mgmt_mail_autogenerated_header branch from cd83274 to d01aec6 Compare June 25, 2026 10:13
@OCA-git-bot OCA-git-bot removed the mod:helpdesk_mgmt Module helpdesk_mgmt label Jun 25, 2026
@jans23
jans23 force-pushed the 18.0-add-helpdesk_mgmt_mail_autogenerated_header branch from d01aec6 to bc8d828 Compare August 12, 2026 11:44
@jans23

jans23 commented Aug 12, 2026

Copy link
Copy Markdown
Contributor Author

@victoralmau I moved the code from #1054 to here as requested.

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

The code seems consistent (although I'm not familiar with mail_autogenerated_header)

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

LGTM

@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@jans23

jans23 commented Aug 18, 2026

Copy link
Copy Markdown
Contributor Author

@pedrobaeza could you merge this or who to ask?

@marcelsavegnago

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1050-by-marcelsavegnago-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@marcelsavegnago The merge process could not be finalized, because command twine upload --disable-progress-bar --non-interactive --repository-url https://upload.pypi.org/legacy/ -u __token__ odoo_addon_helpdesk_mgmt_mail_autogenerated_header-18.0.1.0.0.2-py3-none-any.whl failed with output:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading 
odoo_addon_helpdesk_mgmt_mail_autogenerated_header-18.0.1.0.0.2-py3-none-any.whl
�[33mWARNING �[0m Error during upload. Retry with the --verbose option for more details. 
�[31mERROR   �[0m HTTPError: 429 Too Many Requests from https://upload.pypi.org/legacy/  
         Too Many Requests                                                      

@jans23

jans23 commented Aug 20, 2026

Copy link
Copy Markdown
Contributor Author

@marcelsavegnago could you check and maybe try again? Thanks.

@marcelsavegnago

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-1050-by-marcelsavegnago-bump-nobump, awaiting test results.

@OCA-git-bot

Copy link
Copy Markdown
Contributor

@marcelsavegnago The merge process could not be finalized, because command twine upload --disable-progress-bar --non-interactive --repository-url https://upload.pypi.org/legacy/ -u __token__ odoo_addon_helpdesk_mgmt_mail_autogenerated_header-18.0.1.0.0.2-py3-none-any.whl failed with output:

Uploading distributions to https://upload.pypi.org/legacy/
Uploading 
odoo_addon_helpdesk_mgmt_mail_autogenerated_header-18.0.1.0.0.2-py3-none-any.whl
�[33mWARNING �[0m Error during upload. Retry with the --verbose option for more details. 
�[31mERROR   �[0m HTTPError: 429 Too Many Requests from https://upload.pypi.org/legacy/  
         Too Many Requests                                                      

bt-admin pushed a commit to brain-tec/helpdesk that referenced this pull request Aug 25, 2026
Signed-off-by marcelsavegnago
@jans23

jans23 commented Aug 31, 2026

Copy link
Copy Markdown
Contributor Author

@marcelsavegnago would you try again, please? Or any better idea?

@marcelsavegnago

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

On my way to merge this fine PR!
Prepared branch 18.0-ocabot-merge-pr-1050-by-marcelsavegnago-bump-nobump, awaiting test results.

@OCA-git-bot
OCA-git-bot merged commit 98691d8 into OCA:18.0 Sep 8, 2026
7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

4 participants