[18.0][MIG] website_require_login: Migration to 18.0 by NICO-SOLUTIONS · Pull Request #1077 · OCA/website

NICO-SOLUTIONS · 2024-12-09T15:13:25Z

migration to 18.0

github-actions · 2025-06-08T12:48:15Z

There hasn't been any activity on this pull request in the past 4 months, so it has been marked as stale and it will be closed automatically if no further activity occurs in the next 30 days.
If you want this PR to never become stale, please ask a PSC member to apply the "no stale" label.

NICO-SOLUTIONS · 2025-07-02T14:15:09Z

@OCA/website-maintainers
could you remove the stale tag please?

jguenat

Functional & code review
LGTM

eduezerouali-tecnativa

Code review

@Tecnativa TT58469

pedrobaeza · 2025-10-16T09:32:57Z

/ocabot migration website_require_login
/ocabot merge nobump

OCA-git-bot · 2025-10-16T09:33:04Z

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-1077-by-pedrobaeza-bump-nobump, awaiting test results.

Signed-off-by pedrobaeza

OCA-git-bot · 2025-10-16T09:36:11Z

@pedrobaeza your merge command was aborted due to failed check(s), which you can inspect on this commit of 18.0-ocabot-merge-pr-1077-by-pedrobaeza-bump-nobump.

After fixing the problem, you can re-issue a merge command. Please refrain from merging manually as it will most probably make the target branch red.

NICO-SOLUTIONS · 2025-10-16T09:57:06Z

rebased

pedrobaeza · 2025-10-16T09:59:34Z

Please check CI.

NICO-SOLUTIONS · 2025-10-16T10:45:40Z

@pedrobaeza
I checked the CI.
The failure is coming from website_snippet_country_dropdown (TestUi.test_01_demo_country_dropdown_tour), not from website_require_login.

Anyway, it needs to be addressed before we can merge this one, right?

It seems to be the same issue we’ve seen here:
#1115

pedrobaeza · 2025-10-16T11:08:29Z

Yes, it should be addressed

NICO-SOLUTIONS · 2025-10-16T11:31:08Z

I’m a bit short on time right now... but I’m happy to support and help as much as I can.
@yajo
@ernestotejeda
@CarlosRoca13
@pilarvargas-tecnativa

As you’ve been involved in website_snippet_country_dropdown, would somebody of you be willing to assist in providing a fix for its test? That would be great!

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

[UPD] Update website_require_login.pot [UPD] README.rst Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/ [UPD] README.rst

Translated using Weblate (Spanish) Currently translated at 100.0% (13 of 13 strings) Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/es/ Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-16.0/website-16.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-16-0/website-16-0-website_require_login/

[UPD] Update website_require_login.pot [BOT] post-merge updates Update translation files Updated by "Update PO files to match POT (msgmerge)" hook in Weblate. Translation: website-17.0/website-17.0-website_require_login Translate-URL: https://translation.odoo-community.org/projects/website-17-0/website-17-0-website_require_login/

NICO-SOLUTIONS · 2025-10-17T10:41:26Z

@pilarvargas-tecnativa
thanks a lot pilar! Your quick response and solution is awesome!

pedrobaeza · 2025-10-17T10:44:37Z

/ocabot merge nobump

OCA-git-bot · 2025-10-17T10:44:45Z

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 18.0-ocabot-merge-pr-1077-by-pedrobaeza-bump-nobump, awaiting test results.

OCA-git-bot · 2025-10-17T10:50:55Z

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

NICO-SOLUTIONS · 2025-10-17T10:51:58Z

@pedrobaeza
thx a lot for all your input!

NICO-SOLUTIONS marked this pull request as ready for review December 9, 2024 15:17

NICO-SOLUTIONS mentioned this pull request Dec 9, 2024

Migration to version 18.0 #1062

Open

11 tasks

NICO-SOLUTIONS force-pushed the 18.0-mig-website_require_login branch from ade8ed6 to b1b5e11 Compare February 4, 2025 14:43

github-actions bot added the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jun 8, 2025

github-actions bot removed the stale PR/Issue without recent activity, it'll be soon closed automatically. label Jul 6, 2025

NICO-SOLUTIONS force-pushed the 18.0-mig-website_require_login branch from b1b5e11 to 7078bf9 Compare August 23, 2025 09:08

jguenat approved these changes Oct 15, 2025

View reviewed changes

eduezerouali-tecnativa approved these changes Oct 16, 2025

View reviewed changes

OCA-git-bot added the approved label Oct 16, 2025

OCA-git-bot added this to the 18.0 milestone Oct 16, 2025

OCA-git-bot added a commit that referenced this pull request Oct 16, 2025

Merge PR #1077 into 18.0

30fb34d

Signed-off-by pedrobaeza

OCA-git-bot added the bot is merging ⏳ label Oct 16, 2025

OCA-git-bot removed the bot is merging ⏳ label Oct 16, 2025

NICO-SOLUTIONS force-pushed the 18.0-mig-website_require_login branch from 7078bf9 to ffc149b Compare October 16, 2025 09:48

pilarvargas-tecnativa mentioned this pull request Oct 17, 2025

[18.0][FIX] website_snippet_country_dropdown: Search for menu button by name that never changes #1136

Merged

Aldeigja and others added 5 commits October 17, 2025 12:35

[ADD] website_require_login.

6eb1cfd

[UPD] Update website_require_login.pot

f9dbbad

[UPD] README.rst

68c92f1

Added translation using Weblate (Italian)

4691f0e

Translated using Weblate (Italian)

68600f3

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

hailangvn and others added 6 commits October 17, 2025 12:35

[IMP] website_require_login: pre-commit auto fixes

59706b3

[IMP] website_require_login: pre-commit auto fixes

0f01f8b

[MIG] website_require_login: Migration to 18.0

69a0eda

NICO-SOLUTIONS force-pushed the 18.0-mig-website_require_login branch from ffc149b to 69a0eda Compare October 17, 2025 10:36

OCA-git-bot added the bot is merging ⏳ label Oct 17, 2025

OCA-git-bot merged commit d09b574 into OCA:18.0 Oct 17, 2025
5 of 7 checks passed

OCA-git-bot added merged 🎉 and removed bot is merging ⏳ labels Oct 17, 2025

NICO-SOLUTIONS deleted the 18.0-mig-website_require_login branch October 17, 2025 10:52

Uh oh!

Conversation

NICO-SOLUTIONS commented Dec 9, 2024

Uh oh!

github-actions bot commented Jun 8, 2025

Uh oh!

NICO-SOLUTIONS commented Jul 2, 2025

Uh oh!

jguenat left a comment

Choose a reason for hiding this comment

Uh oh!

eduezerouali-tecnativa left a comment

Choose a reason for hiding this comment

Uh oh!

pedrobaeza commented Oct 16, 2025

Uh oh!

OCA-git-bot commented Oct 16, 2025

Uh oh!

OCA-git-bot commented Oct 16, 2025

Uh oh!

NICO-SOLUTIONS commented Oct 16, 2025

Uh oh!

pedrobaeza commented Oct 16, 2025

Uh oh!

NICO-SOLUTIONS commented Oct 16, 2025

Uh oh!

pedrobaeza commented Oct 16, 2025

Uh oh!

NICO-SOLUTIONS commented Oct 16, 2025

Uh oh!

NICO-SOLUTIONS commented Oct 17, 2025

Uh oh!

pedrobaeza commented Oct 17, 2025

Uh oh!

OCA-git-bot commented Oct 17, 2025

Uh oh!

Uh oh!

OCA-git-bot commented Oct 17, 2025

Uh oh!

NICO-SOLUTIONS commented Oct 17, 2025

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

9 participants