Skip to content

Conversation

@theangryangel
Copy link
Member

@theangryangel theangryangel commented Nov 7, 2025

Building on #36.

Largely standard migration.

Needed to change the unit tests to handle the fact that user_id is not supplied with a default value after odoo/odoo@25a0a72

Pre-commit warnings appear to relate to OCA/oca-addons-repo-template#319.

alexey-pelykh and others added 23 commits November 7, 2025 10:25
Currently translated at 100.0% (9 of 9 strings)

Translation: social-12.0/social-12.0-mail_activity_reminder
Translate-URL: https://translation.odoo-community.org/projects/social-12-0/social-12-0-mail_activity_reminder/pt/
Currently translated at 100.0% (9 of 9 strings)

Translation: social-12.0/social-12.0-mail_activity_reminder
Translate-URL: https://translation.odoo-community.org/projects/social-12-0/social-12-0-mail_activity_reminder/fr/
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_reminder
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_reminder/es/
Currently translated at 100.0% (15 of 15 strings)

Translation: social-16.0/social-16.0-mail_activity_reminder
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_activity_reminder/it/
Currently translated at 100.0% (16 of 16 strings)

Translation: social-17.0/social-17.0-mail_activity_reminder
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_activity_reminder/it/
@theangryangel theangryangel force-pushed the 19.0-mig-mail_activity_reminder branch from 6f72c00 to a547a36 Compare November 7, 2025 11:18
@theangryangel theangryangel marked this pull request as ready for review November 7, 2025 11:21
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!

@OCA-git-bot
Copy link
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). 🤖

@StefanRijnhart
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-109-by-StefanRijnhart-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit d14b834 into OCA:19.0 Nov 21, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@StefanRijnhart
Copy link
Member

/ocabot migration mail_activity_reminder

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Nov 21, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 21, 2025
32 tasks
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.