Skip to content

Conversation

@tishmen
Copy link

@tishmen tishmen commented Oct 19, 2025

Scope

  • mail_debranding

Depends on

  • None

Summary

  • Import mail_debrand history from OCA/mail 18.0 using git filter-repo (isolated path + rename to mail_debranding) and git am (authors/subjects preserved).
  • Migrate to Odoo 19; set version to 19.0.1.0.0; keep depends=["mail"].
  • Adjust any template/rendering changes for 19.0 if needed.

Temporary references

  • None

Pre-commit

  • Ran locally; auto-fixes committed separately as [IMP].

Tests

  • Local: ./odoo/odoo-bin -c odoo.conf -i mail_debranding --test-enable --log-level=info --stop-after-init
  • Notes: Verify mail notification and signup templates render; ensure “Powered by” link removal still holds.

rven and others added 30 commits October 19, 2025 15:13
Currently translated at 20.0% (1 of 5 strings)

Translation: social-13.0/social-13.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-13-0/social-13-0-mail_debrand/nl/
Currently translated at 100.0% (5 of 5 strings)

Translation: social-13.0/social-13.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-13-0/social-13-0-mail_debrand/fr/
Currently translated at 100.0% (5 of 5 strings)

Translation: social-13.0/social-13.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-13-0/social-13-0-mail_debrand/fr/
There's no commits preservation, as the module doesn't serve as is on v9.
Currently translated at 80.0% (4 of 5 strings)

Translation: social-14.0/social-14.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-14-0/social-14-0-mail_debrand/fr_FR/
Currently translated at 80.0% (4 of 5 strings)

Translation: social-14.0/social-14.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-14-0/social-14-0-mail_debrand/it/
nguyenminhchien and others added 17 commits October 19, 2025 15:13
Currently translated at 100.0% (3 of 3 strings)

Translation: social-16.0/social-16.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-16-0/social-16-0-mail_debrand/fr/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: social-17.0/social-17.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_debrand/
Currently translated at 100.0% (3 of 3 strings)

Translation: social-17.0/social-17.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/social-17-0/social-17-0-mail_debrand/it/
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: mail-18.0/mail-18.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_debrand/
See odoo/odoo@9f65f1a

Fixes
```
2025-02-12 16:01:58,174 254 ERROR odoo odoo.addons.mail_debrand.tests.test_mail_debrand: FAIL: TestMailDebrand.test_default_debrand
Traceback (most recent call last):
  File "/__w/mail/mail/mail_debrand/tests/test_mail_debrand.py", line 52, in test_default_debrand
    self._test_debrand_by_lang(
  File "/__w/mail/mail/mail_debrand/tests/test_mail_debrand.py", line 47, in _test_debrand_by_lang
    self.assertIn(term, body)
AssertionError: 'Powered by' not found in Markup('\n<html>\n<head>\n
...
```
Currently translated at 100.0% (3 of 3 strings)

Translation: mail-18.0/mail-18.0-mail_debrand
Translate-URL: https://translation.odoo-community.org/projects/mail-18-0/mail-18-0-mail_debrand/tr/
@tishmen tishmen marked this pull request as ready for review October 19, 2025 13:59
@tishmen tishmen force-pushed the 19.0-mig-mail_debranding branch from f629b8c to c0a183e Compare October 19, 2025 14:01
@tishmen
Copy link
Author

tishmen commented Oct 19, 2025

Hi @pedrobaeza, mail_debranding migration is ready for review.

  • Imported full history from OCA/mail 18.0, renamed across history mail_debrand → mail_debranding using git filter-repo, then applied with git am to preserve authors/subjects.
  • Migrated to Odoo 19; manifest set to 19.0.1.0.0; no extra deps beyond mail.
  • 19.0 deltas applied in tests: use odoo.tools.urls.urljoin; switch to self.env._; replace deprecated toggle_active with action_unarchive.
  • Commit hygiene: first [IMP] pre-commit auto-fixes, then single [MIG] commit (squashed).

Pre-commit

  • Ran locally; clean.

Tests

  • Local run: ./odoo/odoo-bin -c odoo.conf -i mail_debranding --test-enable --log-level=info --stop-after-init

Please let me know if you want any additional adjustments. Thank you!

@minhthie
Copy link

minhthie commented Nov 9, 2025

Functional test looks good.

image

However, the phrase "Powered by Odoo" still appears on the login page. I know the module is about mail debranding but maybe it's good to handle them all at once.

image

@tishmen
Copy link
Author

tishmen commented Nov 9, 2025

@minhthie That functionality is covered in the following pull request: #109

Copy link

@RyanTranAVB RyanTranAVB left a comment

Choose a reason for hiding this comment

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

LGTM, but do remove the comment before merging

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.

Looking good! Care to fix that outdated comment?

@tishmen
Copy link
Author

tishmen commented Dec 21, 2025

Sorry for the delay — I’ve updated the PR and removed the migration-only comment. Thanks for the review!

@StefanRijnhart
Copy link
Member

Thanks! Please squash that commit into the migration commit (which introduces the comment removed in the last commit).

@tishmen tishmen force-pushed the 19.0-mig-mail_debranding branch from 58746a7 to e912dc0 Compare December 21, 2025 23:37
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.