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

[17.0][MIG] website_snippet_marginless_gallery: Migration to version 17.0 #1074

Merged

Conversation

pilarvargas-tecnativa
Copy link

cc @Tecnativa TT51978

@chienandalu @CarlosRoca13 please review

Jairo Llopis and others added 16 commits November 29, 2024 15:13
Credit creators, using same name across modules to avoid split statistics.

Ensure images get displayed if content is removed.
- Relicense to LGPL.
- Move tour to demo asset.
- Remove precompiled CSS.
- Update snippet options JS API.
- Update tour's API and definition.
Currently translated at 100.0% (5 of 5 strings)

Translation: website-12.0/website-12.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-12-0/website-12-0-website_snippet_marginless_gallery/fr/
Currently translated at 100.0% (5 of 5 strings)

Translation: website-12.0/website-12.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-12-0/website-12-0-website_snippet_marginless_gallery/pt_BR/
Currently translated at 100.0% (5 of 5 strings)

Translation: website-12.0/website-12.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-12-0/website-12-0-website_snippet_marginless_gallery/es/
Currently translated at 100.0% (5 of 5 strings)

Translation: website-12.0/website-12.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-12-0/website-12-0-website_snippet_marginless_gallery/ca/
Currently translated at 80.0% (4 of 5 strings)

Translation: website-15.0/website-15.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_snippet_marginless_gallery/de/
Currently translated at 100.0% (5 of 5 strings)

Translation: website-15.0/website-15.0-website_snippet_marginless_gallery
Translate-URL: https://translation.odoo-community.org/projects/website-15-0/website-15-0-website_snippet_marginless_gallery/it/
@pedrobaeza
Copy link
Member

/ocabot migration website_snippet_marginless_gallery

@OCA-git-bot OCA-git-bot added this to the 17.0 milestone Dec 2, 2024
@OCA-git-bot OCA-git-bot mentioned this pull request Dec 2, 2024
15 tasks
@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 17.0-mig-website_snippet_marginless_gallery branch 3 times, most recently from 9f209c5 to b8dc87c Compare December 2, 2024 12:16
@pilarvargas-tecnativa
Copy link
Author

ping @chienandalu

Copy link
Member

@chienandalu chienandalu left a comment

Choose a reason for hiding this comment

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

Tested 👍

@pilarvargas-tecnativa
Copy link
Author

ping @juancarlosonate-tecnativa Can you take a look?

@juancarlosonate-tecnativa

Minimal change: in USAGE.md it says that the block is in the features category when it is in structure

@pilarvargas-tecnativa pilarvargas-tecnativa force-pushed the 17.0-mig-website_snippet_marginless_gallery branch from b8dc87c to 5c27c36 Compare January 10, 2025 11:35
@pilarvargas-tecnativa
Copy link
Author

Minimal change: in USAGE.md it says that the block is in the features category when it is in structure

Done, thanks! ;)

Copy link

@juancarlosonate-tecnativa juancarlosonate-tecnativa left a comment

Choose a reason for hiding this comment

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

LGTM

@pedrobaeza
Copy link
Member

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

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

@OCA-git-bot OCA-git-bot merged commit d5421e7 into OCA:17.0 Jan 10, 2025
7 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza pedrobaeza deleted the 17.0-mig-website_snippet_marginless_gallery branch January 10, 2025 11:57
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.