Skip to content

Conversation

@P-H-Phuc
Copy link

This module was initially part of the Foodcoop set of addons. We're now migrating it to 18.0 and taking this occasion to contribute it to OCA.

Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Nice, originally a port of OCA/social#521 I think. Can you squash the clean up commits a little bit and make sure the module name is present in all of the commit messages?

@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-mail_template_conditional_attachment branch 2 times, most recently from 435c3cc to c4309d6 Compare November 6, 2025 05:10
@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-mail_template_conditional_attachment branch 3 times, most recently from 6b82476 to fcaf162 Compare November 6, 2025 05:23
Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks for the updates! Looking good now, only one remaining suggestion.

@P-H-Phuc P-H-Phuc force-pushed the 18.0-mig-mail_template_conditional_attachment branch from fcaf162 to 036a40c Compare November 6, 2025 09:55
Copy link
Member

@StefanRijnhart StefanRijnhart left a comment

Choose a reason for hiding this comment

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

Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants