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

[11.0] [FIX] l10n_it_central_journal: fix query for ambiguous column #2114

Merged
merged 1 commit into from
Mar 16, 2021

Conversation

rcerrillo2
Copy link

Descrizione del problema o della funzionalità:
Il campo journal_id è in account_move e anche in account_move_line, per questo quando el girornale viene stampato, c'e un errore

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

Copy link
Member

@eLBati eLBati left a comment

Choose a reason for hiding this comment

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

Sulla 12 è già così

@sergiocorato
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 11.0-ocabot-merge-pr-2114-by-sergiocorato-bump-patch, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Feb 13, 2021
Signed-off-by sergiocorato
@OCA-git-bot
Copy link
Contributor

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

@alessandrocamilli
Copy link
Contributor

@sergiocorato @eLBati perchè il bot non è riuscito a mergiare?

@eLBati
Copy link
Member

eLBati commented Feb 15, 2021

Sembra sempre per #1921 ma è molto strano

@SimoRubi
Copy link
Member

Potrebbe essere perché 11.0 è rosso di suo

@sergiocorato
Copy link
Contributor

Potrebbe essere perché 11.0 è rosso di suo

vero, da quanto è in rosso?

@sergiocorato
Copy link
Contributor

questo sembra l'errore no? https://travis-ci.com/github/OCA/l10n-italy/jobs/481409026#L2006

@SimoRubi
Copy link
Member

Potrebbe essere perché 11.0 è rosso di suo

vero, da quanto è in rosso?

Credo sia dal merge manuale di #1969 di settimana scorsa

@rcerrillo2
Copy link
Author

Se devo o posso fare qualcosa ditemi, perchè non vedo il problema di forma chiara

@primes2h
Copy link
Contributor

primes2h commented Feb 15, 2021

Sembra sempre per #1921 ma è molto strano

A me sembra che il problema sia legato alla #1969.
Anche in questo caso ha creato molti commit per il .pot di l10n_it_account_balance_report e per ciascuno lancia una build di runbot.
https://runbot.odoo-community.org/runbot/branch/2461418

Ho come l'impressione che dopo un po' vada in timeout perché, mentre sta lavorando sulla n-sima build, quella iniziale del commit di merge della PR viene eliminata, bloccando il processo. (successivamente il sistema "killa" le altre build).
C'è per caso un limite massimo di build runbot attive contemporanee?

@eLBati
Copy link
Member

eLBati commented Feb 17, 2021

Sarebbe da chiedere a OCA ad esempio qui https://discord.gg/QtFwCekZk7

@OpenCode
Copy link
Contributor

@simahawk
Copy link
Contributor

simahawk commented Mar 1, 2021

Can you fix 11.0 first in another PR? I see https://travis-ci.com/github/OCA/l10n-italy/jobs/481409026

@OpenCode
Copy link
Contributor

OpenCode commented Mar 2, 2021

Can you fix 11.0 first in another PR? I see https://travis-ci.com/github/OCA/l10n-italy/jobs/481409026

Ciao. Il problema è capire dove si necessita del fix.

@eLBati
Copy link
Member

eLBati commented Mar 8, 2021

#1921

@OpenCode
Copy link
Contributor

@rcerrillo2 Potresti fare il rebase?

@rcerrillo2
Copy link
Author

@rcerrillo2 Potresti fare il rebase?

Fatto, vedremo se funziona adesso

@OpenCode
Copy link
Contributor

/ocabot merge patch

@OCA-git-bot
Copy link
Contributor

On my way to merge this fine PR!
Prepared branch 11.0-ocabot-merge-pr-2114-by-OpenCode-bump-patch, awaiting test results.

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

@OCA-git-bot OCA-git-bot merged commit 13a9c5a into OCA:11.0 Mar 16, 2021
@OCA-git-bot
Copy link
Contributor

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

9 participants