Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[16.0][MIG] website_legal_page: migration to 16.0 #946

Merged
merged 51 commits into from
Mar 3, 2023

Conversation

miguel-S73
Copy link
Contributor

Standard migration

AungKoKoLin1997
AungKoKoLin1997 approved these changes Jan 31, 2023
website_legal_page/README.rst Outdated Show resolved Hide resolved
@pilarvargas-tecnativa
Copy link

ping @CarlosRoca13

Antonio Espinosa and others added 25 commits March 3, 2023 08:26
- Fix review comments
- Icon changed
- Bug tracker added
- [IMP] README and OCA as author

[FIX] Space at the end of a traslation line

[FIX] mailto links in default legal pages is malformed
Many submodules from this one need the template I'm adding.

Having it here allows for better reusability and avoids duplicated translation efforts.
* Move legal pages to explicit /legal routes

Fixes by PR review

Fix my obvious lack of testing
- Update copyright headers.
- Update copyright holders data.
- Remove .pot file.
- Add migration script.
* [ADD][website_multi_theme] Allow different themes by website

This a somewhat hacky addon that adds support for setting a different theme by website.

Always updates views arch in development and demo instances.

This allows faster development, while keeping stability and no surprises in production.

* fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! fixup! fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! fixup! fixup! fixup! fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* [FIX][website_legal_page] Correctly raise a 404 error instead of a 500 if a view is not found

* fixup! fixup! fixup! fixup! fixup! fixup! fixup! [ADD][website_multi_theme] Allow different themes by website

* fixup! [FIX][website_legal_page] Correctly raise a 404 error instead of a 500 if a view is not found

* squash! fixup! [FIX][website_legal_page] Correctly raise a 404 error instead of a 500 if a view is not found

Fix tests that were getting warnings logged and random timeouts.
[FIX] Linter on website_multi_theme py file
[FIX] website_legal_page: Fix PO
Currently translated at 100,0% (49 of 49 strings)

Translation: website-11.0/website-11.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-11-0/website-11-0-website_legal_page/de/
Currently translated at 91,8% (45 of 49 strings)

Translation: website-11.0/website-11.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-11-0/website-11-0-website_legal_page/es/
Haftec and others added 15 commits March 3, 2023 08:26
Currently translated at 7.8% (4 of 51 strings)

Translation: website-14.0/website-14.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-14-0/website-14-0-website_legal_page/ca/
Currently translated at 100.0% (51 of 51 strings)

Translation: website-14.0/website-14.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-14-0/website-14-0-website_legal_page/it/
Currently translated at 98.0% (50 of 51 strings)

Translation: website-15.0/website-15.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_legal_page/de/
Remove redirection of /terms to /legal but maintain module to
complement account terms or to set legal terms to pages that
has not account installed.
Updated by "Update PO files to match POT (msgmerge)" hook in Weblate.

Translation: website-15.0/website-15.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_legal_page/
@miguel-S73
Copy link
Contributor Author

I have updated my PR with the last changes in this module in odoo 15.0

@miguel-S73
Copy link
Contributor Author

@CarlosRoca13
How do you see last changes?

@pedrobaeza
Copy link
Member

Finally this was polished in v15, and transferred here, so I think this can cover a specific use case. Merging then:

/ocabot migration website_legal_page
/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

@OCA-git-bot OCA-git-bot added this to the 16.0 milestone Mar 3, 2023
@OCA-git-bot OCA-git-bot merged commit 857416a into OCA:16.0 Mar 3, 2023
@OCA-git-bot
Copy link
Contributor

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

@miguel-S73 miguel-S73 deleted the 16.0-mig-website_legal_page branch March 3, 2023 08:58
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.