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

[15.0][MIG] website_legal_page: Migration to 15.0 #897

Merged
merged 40 commits into from
May 26, 2022

Conversation

CarlosRoca13
Copy link
Contributor

cc @Tecnativa TT36594

This PR superseds this other #875 as the author is giving no response, but his commits are preserved

please @pedrobaeza @Tardo review this

Antonio Espinosa and others added 30 commits May 25, 2022 16:47
- 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/
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/
IMP  website_legal_page with CSS classes to ease styling
FIX README
[FIX] website_legal_page: update view names
Currently translated at 100.0% (49 of 49 strings)

Translation: website-12.0/website-12.0-website_legal_page
Translate-URL: https://translation.odoo-community.org/projects/website-12-0/website-12-0-website_legal_page/pt_BR/
@CarlosRoca13
Copy link
Contributor Author

Copy link
Member

@ioans73 ioans73 left a comment

Choose a reason for hiding this comment

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

Functional review

@pedrobaeza
Copy link
Member

/ocabot merge nobump
/ocabot migration website_legal_page

@OCA-git-bot OCA-git-bot added this to the 15.0 milestone May 26, 2022
@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 15.0-ocabot-merge-pr-897-by-pedrobaeza-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot mentioned this pull request May 26, 2022
12 tasks
@OCA-git-bot OCA-git-bot merged commit ccf9155 into OCA:15.0 May 26, 2022
@OCA-git-bot
Copy link
Contributor

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