Skip to content

[18.0] [MIG] l10n it_amount_to_text#4989

Merged
OCA-git-bot merged 5 commits into
OCA:18.0from
PNLUG:18.0-mig-l10n_it_amount_to_text
Feb 20, 2026
Merged

[18.0] [MIG] l10n it_amount_to_text#4989
OCA-git-bot merged 5 commits into
OCA:18.0from
PNLUG:18.0-mig-l10n_it_amount_to_text

Conversation

@primes2h

@primes2h primes2h commented Nov 7, 2025

Copy link
Copy Markdown
Contributor

No description provided.

@primes2h primes2h mentioned this pull request Nov 7, 2025
Open
46 tasks
@tafaRU

tafaRU commented Nov 7, 2025

Copy link
Copy Markdown
Member

/ocabot migration l10n_it_amount_to_text

@OCA-git-bot OCA-git-bot added this to the 18.0 milestone Nov 7, 2025
@primes2h primes2h force-pushed the 18.0-mig-l10n_it_amount_to_text branch 2 times, most recently from 4ebd250 to 845e903 Compare November 7, 2025 16:49
@primes2h primes2h force-pushed the 18.0-mig-l10n_it_amount_to_text branch from 845e903 to 43d74a8 Compare November 19, 2025 15:35
@monen17

monen17 commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

/ocabot rebase

@OCA-git-bot

OCA-git-bot commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

@monen17 The rebase process failed, because command git push --force PNLUG tmp-pr-4989:18.0-mig-l10n_it_amount_to_text failed with output:

remote: Permission to PNLUG/l10n-italy.git denied to OCA-git-bot.
fatal: unable to access 'https://github.com/PNLUG/l10n-italy/': The requested URL returned error: 403

@monen17

monen17 commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

@primes2h potresti fare un rebase please?

@primes2h primes2h force-pushed the 18.0-mig-l10n_it_amount_to_text branch from 43d74a8 to e7ca5c3 Compare February 20, 2026 15:19
@primes2h

primes2h commented Feb 20, 2026

Copy link
Copy Markdown
Contributor Author

@primes2h potresti fare un rebase please?

Fatto.

monen17
monen17 approved these changes Feb 20, 2026
View reviewed changes

@monen17 monen17 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.

Ho provato e fatto revisione del codice, per me è ok, grazie!
Prima: Image
Dopo: Image

Comment thread l10n_it_amount_to_text/tests/test_amount_to_text.py
@primes2h

primes2h commented Feb 20, 2026

Copy link
Copy Markdown
Contributor Author

Ho provato e fatto revisione del codice, per me è ok, grazie! Prima: Image Dopo: Image

Grazie della review!

@eLBati

eLBati commented Feb 20, 2026

Copy link
Copy Markdown
Member

/ocabot merge nobump

@OCA-git-bot

OCA-git-bot commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

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

@OCA-git-bot

OCA-git-bot commented Feb 20, 2026

Copy link
Copy Markdown
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 5c4ffc1 into OCA:18.0 Feb 20, 2026
7 checks passed
@OCA-git-bot

OCA-git-bot commented Feb 20, 2026

Copy link
Copy Markdown
Contributor

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

Reviewers

@eLBati eLBati eLBati approved these changes

@monen17 monen17 monen17 approved these changes

Assignees

No one assigned

Labels

approved merged 🎉 no issue needed ready to merge

Projects

None yet

Milestone

18.0

Development

Successfully merging this pull request may close these issues.

5 participants

@primes2h @tafaRU @monen17 @OCA-git-bot @eLBati
⚡