Skip to content

Conversation

@RabbitJon-S73
Copy link

Standard migration to 19.0

Copy link

@sbiosca-s73 sbiosca-s73 left a comment

Choose a reason for hiding this comment

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

LGTM!

@Reyes4711-S73
Copy link

@OCA/server-environment-maintainers please, can you merge this PR?

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

Copy link

@Jortolsa-S73 Jortolsa-S73 left a comment

Choose a reason for hiding this comment

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

LGTM

@RabbitJon-S73
Copy link
Author

@thomaspaulb hi! Could you validate the migration?

@thomaspaulb
Copy link

/ocabot migration bus_alt_connection

/ocabot merge nobump

@OCA-git-bot OCA-git-bot added this to the 19.0 milestone Nov 11, 2025
@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 19.0-ocabot-merge-pr-3399-by-thomaspaulb-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 11, 2025
Signed-off-by thomaspaulb
@OCA-git-bot OCA-git-bot mentioned this pull request Nov 11, 2025
40 tasks
@OCA-git-bot
Copy link
Contributor

@thomaspaulb your merge command was aborted due to failed check(s), which you can inspect on this commit of 19.0-ocabot-merge-pr-3399-by-thomaspaulb-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.

@thomaspaulb
Copy link

@RabbitJon-S73 Do you get this click-contrib-make-pot error on your local also?

@RabbitJon-S73
Copy link
Author

@RabbitJon-S73 Do you get this click-contrib-make-pot error on your local also?

No, it didn't happen to me. I'm not sure about what I have to do. Could you give me a hint? @thomaspaulb

@thomaspaulb
Copy link

@sbidoul It looks like this is the problem:

image

After which lang_export.data pops up as False and there is the bool error here

I can reproduce on my local if I install bus_alt_connection on an otherwise empty demo database and then run

click-odoo-makepot --addons-dir auto/addons -d $DATABASE --msgmerge-if-new-pot --commit --log-level=debug

Is this a known issue? I don't see it in https://github.com/acsone/click-odoo-contrib/issues

@sbidoul
Copy link
Member

sbidoul commented Nov 11, 2025

This is being handled here: acsone/click-odoo-contrib#173

@sbidoul
Copy link
Member

sbidoul commented Nov 11, 2025

click-odoo-contrib 1.23.1 has been released with a fix for empty .pot files in Odoo 19. Please retry the merge.

@thomaspaulb
Copy link

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 19.0-ocabot-merge-pr-3399-by-thomaspaulb-bump-nobump, awaiting test results.

@thomaspaulb
Copy link

Thank you @sbidoul

@OCA-git-bot OCA-git-bot merged commit 4962e7a into OCA:19.0 Nov 11, 2025
6 of 7 checks passed
@OCA-git-bot
Copy link
Contributor

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