Skip to content

[18.0] porting currency_rate_update_boi#4823

Merged
OCA-git-bot merged 20 commits into
OCA:18.0from
eLBati:18.0-mig-currency_rate_update_boi
Sep 12, 2025
Merged

[18.0] porting currency_rate_update_boi#4823
OCA-git-bot merged 20 commits into
OCA:18.0from
eLBati:18.0-mig-currency_rate_update_boi

Conversation

@eLBati

@eLBati eLBati commented Jul 7, 2025

Copy link
Copy Markdown
Member

No description provided.

eLBati and others added 18 commits July 7, 2025 12:15
Currently translated at 100.0% (4 of 4 strings)

Translation: l10n-italy-12.0/l10n-italy-12.0-currency_rate_update_boi
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-12-0/l10n-italy-12-0-currency_rate_update_boi/it/
Pre-commit error is:
Error: website key not found in manifest in currency_rate_update_boi.
Currently translated at 100.0% (6 of 6 strings)

Translation: l10n-italy-14.0/l10n-italy-14.0-currency_rate_update_boi
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-14-0/l10n-italy-14-0-currency_rate_update_boi/it/
Currently translated at 100.0% (6 of 6 strings)

Translation: l10n-italy-16.0/l10n-italy-16.0-currency_rate_update_boi
Translate-URL: https://translation.odoo-community.org/projects/l10n-italy-16-0/l10n-italy-16-0-currency_rate_update_boi/it/
@eLBati

eLBati commented Jul 7, 2025

Copy link
Copy Markdown
Member Author

/ocabot migration currency_rate_update_boi

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Jul 7, 2025
@OCA-git-bot OCA-git-bot mentioned this pull request Jul 7, 2025
46 tasks
@eLBati

eLBati commented Jul 14, 2025

Copy link
Copy Markdown
Member Author

#3801 (comment)

@andreampiovesana

Copy link
Copy Markdown
Contributor

@eLBati sei riuscito a vedere i test?

@TheMule71

TheMule71 commented Aug 1, 2025

Copy link
Copy Markdown
Contributor

@eLBati eLBati force-pushed the 18.0-mig-currency_rate_update_boi branch from 7534cfc to 31d52ee Compare August 29, 2025 08:20
Actually use mock data to avoid external call.

Remove useless tests, as they test standard mechanisms.
Rename currency_rate_update_boi to l10n_it_currency_rate_update_boi

@TheMule71 TheMule71 left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM
(non ho potuto provare la migrazione dalla 16)

@andreampiovesana andreampiovesana left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM

@eLBati

eLBati commented Sep 12, 2025

Copy link
Copy Markdown
Member Author

/ocabot merge nobump

@OCA-git-bot

Copy link
Copy Markdown
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 18.0-ocabot-merge-pr-4823-by-eLBati-bump-nobump, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit ac6f0f7 into OCA:18.0 Sep 12, 2025
6 of 7 checks passed
@OCA-git-bot

Copy link
Copy Markdown
Contributor

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

@francesco-ooops

Copy link
Copy Markdown
Contributor

@eLBati penso includere questi possa semplificare la vita: #4894

@eLBati

eLBati commented Sep 12, 2025

Copy link
Copy Markdown
Member Author

@francesco-ooops c'è: bff8940

@francesco-ooops

Copy link
Copy Markdown
Contributor

@eLBati grazie, avevo visto male!

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.