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

[14.0][OU-FIX] website: fix extract_footer_copyright_company_name #4245

Merged

Conversation

pilarvargas-tecnativa
Copy link
Contributor

Fix the way to get the views

cc @Tecnativa TT44601

@chienandalu @pedrobaeza please review

@OCA-git-bot
Copy link
Contributor

Hi @StefanRijnhart, @MiquelRForgeFlow, @pedrobaeza,
some modules you are maintaining are being modified, check this out!

@pedrobaeza pedrobaeza added this to the 14.0 milestone Nov 29, 2023
@pedrobaeza
Copy link
Member

The same about the commit message as in #4244

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 14.0-fix-website-copyright_company_name branch from aaabff9 to 53e3661 Compare November 30, 2023 06:50
@pilarvargas-tecnativa
Copy link
Contributor Author

The same about the commit message as in #4244

Thanks

@pedrobaeza
Copy link
Member

Check about the first line of the commit message and the sub command I mentioned in other pull request.

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 14.0-fix-website-copyright_company_name branch from 53e3661 to c52e34c Compare November 30, 2023 07:22
Previous code was ignoring possible customizations on the copyright information,
as it took it from the master template. With this change, the information is
taken from the customized template if exists, or the master otherwise, using
the existing CoW mechanism for getting it in a simple way.

TT44601
@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 14.0-fix-website-copyright_company_name branch from c52e34c to c489fec Compare November 30, 2023 07:27
@pilarvargas-tecnativa
Copy link
Contributor Author

Check about the first line of the commit message and the sub command I mentioned in other pull request.

Done!

@pedrobaeza pedrobaeza merged commit f903e1a into OCA:14.0 Nov 30, 2023
2 checks passed
@pedrobaeza pedrobaeza deleted the 14.0-fix-website-copyright_company_name branch November 30, 2023 07:35
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.

3 participants