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

[12.0][IMP] l10n_es_ticketbai_pos: Round line quantity to 8 decimals when s… #3178

Conversation

ao-landoo
Copy link
Contributor

…ending to tax agency

Cherry-pick de #3177

@ao-landoo
Copy link
Contributor Author

Mergeo cherry-pick

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 4, 2023
Signed-off-by ao-landoo
@OCA-git-bot
Copy link
Contributor

@ao-landoo your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch.

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.

@ao-landoo
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 7, 2023
Signed-off-by ao-landoo
@OCA-git-bot
Copy link
Contributor

@ao-landoo your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch.

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.

@ao-landoo
Copy link
Contributor Author

@pedrobaeza Por el error me da que es de conexión, pero siempre está fallando el de OCB. ¿Sabes cual puede ser la razón?

@pedrobaeza
Copy link
Member

Pues está siendo pura cuestión de azar. Acabo de reintentar la del PR y ya ha salido en verde. Prueba una vez más a ver.

@ao-landoo
Copy link
Contributor Author

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Aug 7, 2023
Signed-off-by ao-landoo
@OCA-git-bot
Copy link
Contributor

@ao-landoo your merge command was aborted due to failed check(s), which you can inspect on this commit of 12.0-ocabot-merge-pr-3178-by-ao-landoo-bump-patch.

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.

@pedrobaeza
Copy link
Member

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 12.0-ocabot-merge-pr-3178-by-pedrobaeza-bump-patch, awaiting test results.

@OCA-git-bot OCA-git-bot merged commit 11c901d into OCA:12.0 Aug 7, 2023
6 checks passed
@OCA-git-bot
Copy link
Contributor

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

@pedrobaeza
Copy link
Member

Contigo se niega 🤣

@pedrobaeza pedrobaeza added this to the 12.0 milestone Aug 7, 2023
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.

3 participants