Skip to content

[19.0][MIG]: website_form_require_legal#1130

Merged
OCA-git-bot merged 19 commits intoOCA:19.0from
Studio73:19.0-mig-website_form_require_legal
Oct 13, 2025
Merged

[19.0][MIG]: website_form_require_legal#1130
OCA-git-bot merged 19 commits intoOCA:19.0from
Studio73:19.0-mig-website_form_require_legal

Conversation

@Alexgars73
Copy link
Contributor

No description provided.

CarlosRoca13 and others added 18 commits October 3, 2025 12:11
Currently translated at 100.0% (4 of 4 strings)

Translation: website-15.0/website-15.0-website_form_require_legal
Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_form_require_legal/de/
TT44376

[UPD] Update website_form_require_legal.pot

Update translation files

Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: website-16.0/website-16.0-website_form_require_legal
Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_form_require_legal/
Translated using Weblate (Italian)

Currently translated at 100.0% (4 of 4 strings)

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

Translation: website-17.0/website-17.0-website_form_require_legal
Translate-URL: https://translation.odoo-community.org/projects/website-17-0/website-17-0-website_form_require_legal/
@Alexgars73
Copy link
Contributor Author

Hello @CarlosRoca13 @pilarvargas-tecnativa if you can check it.

#. odoo-javascript
#: code:addons/website_form_require_legal/static/src/xml/website_form_editor.xml:0
msgid "terms and conditions"
msgstr ""
Copy link
Contributor

Choose a reason for hiding this comment

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

Why are you deleting it?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Is it necessary? There is no traduction.

Copy link
Member

Choose a reason for hiding this comment

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

I think this file is generated by Weblate so it can be removed. Correct me if I'm wrong

Copy link
Contributor

Choose a reason for hiding this comment

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

Correct, Weblate is responsible for updating the translation files, so it should not be deleted, as this only adds an unnecessary diff to the migration commit.
At most, regenerate the .pot and update the translations if necessary.
The .pot file is responsible for generating the rest of the translation files in different languages *.po.

Copy link
Member

Choose a reason for hiding this comment

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

@pilarvargas-tecnativa Alright, thanks for your clarification! Can you please recover the file please, @Alexgars73

Copy link
Contributor

@CarlosRoca13 CarlosRoca13 left a comment

Choose a reason for hiding this comment

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

Tested on runbot

Copy link
Contributor

@pilarvargas-tecnativa pilarvargas-tecnativa left a comment

Choose a reason for hiding this comment

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

Restore the .pot file, please.

@Alexgars73 Alexgars73 force-pushed the 19.0-mig-website_form_require_legal branch from 9241470 to 9f665f5 Compare October 13, 2025 08:55
@Alexgars73 Alexgars73 force-pushed the 19.0-mig-website_form_require_legal branch from 9f665f5 to e6024ad Compare October 13, 2025 08:56
@Alexgars73
Copy link
Contributor Author

@pilarvargas-tecnativa Done.

@pilarvargas-tecnativa
Copy link
Contributor

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

@pedrobaeza
Copy link
Member

/ocabot migration website_form_require_legal
/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Oct 13, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Oct 13, 2025
10 tasks
@OCA-git-bot OCA-git-bot merged commit 319f1c3 into OCA:19.0 Oct 13, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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