Skip to content

[18.0][ADD] attachment_queue_email - #3727

Open
sebalix wants to merge 16 commits into
OCA:18.0from
akretion:18.0-mig-attachment_queue_email
Open

sebalix wants to merge 16 commits into
OCA:18.0from
akretion:18.0-mig-attachment_queue_email

Conversation

@sebalix

@sebalix sebalix commented Sep 15, 2026

Copy link
Copy Markdown
Contributor

Abstract module for importing emails attachments.

Each email's attachment matching a given "Attachment Condition" will be imported creating a new attachment.queue object. These attachment.queue objects are files wrapped with additional fields (mainly a Filed Type and a State) making them ready to be processed by a custom module as you can read in the
attachment_queue documentation.

Migrated from our fork https://github.com/akretion/server-tools/commits/16.0-mig-attachment_queue_email/

@OCA-git-bot OCA-git-bot added series:18.0 mod:attachment_queue_email Module attachment_queue_email labels Sep 15, 2026
@sebalix sebalix changed the title [18.0][MIG] attachment_queue_email [18.0][ADD] attachment_queue_email Sep 15, 2026
@sebalix
sebalix force-pushed the 18.0-mig-attachment_queue_email branch from ae4522f to 1273a97 Compare September 15, 2026 14:48
@sebalix
sebalix marked this pull request as ready for review September 15, 2026 14:59

@metaminux metaminux left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Hi @sebalix

Some minor changes... only the column_invisible seems really needed.

Comment thread attachment_queue_email/views/fetchmail_attachment_condition.xml Outdated
Comment thread attachment_queue_email/readme/DESCRIPTION.md Outdated
Comment thread attachment_queue_email/security/ir.model.access.csv Outdated
Comment thread attachment_queue_email/models/fetchmail_attachment_condition.py Outdated
Comment thread attachment_queue_email/__manifest__.py Outdated
@@ -0,0 +1,24 @@
# @author Sébastien BEAU @ Akretion
# @author Florian DA COSTA @ Akretion
# @author Benoit GUILLOT @ Akretion

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

It seems this code originated from the file_email module proposed in version 8.0 before being renamed in 12.0.

I suggest adding people involved in CONTRIBUTORS.md (@bealdav and @bguillot are missing, maybe others...)

Suggested change
# @author Benoit GUILLOT @ Akretion
# Copyright 2012 Akretion

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

I could not find any mention of @bealdav in code or commits 🤷‍♂️

@sebalix
sebalix force-pushed the 18.0-mig-attachment_queue_email branch from 1273a97 to 8f96ae6 Compare September 16, 2026 07:28
@sebalix
sebalix requested a review from metaminux September 16, 2026 07:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

mod:attachment_queue_email Module attachment_queue_email series:18.0

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants