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.

Sign up for GitHub

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

Jump to bottom

[11.0] New module l10n_it_account_balance_report #1282

Closed
wants to merge 1 commit into from
Closed

[11.0] New module l10n_it_account_balance_report #1282

wants to merge 1 commit into from

Conversation

SilvioGregorini
Copy link
Contributor

Descrizione del problema o della funzionalità: aggiunti report .pdf e .xls a sezioni contrapposte per conto economico e stato patrimoniale

Comportamento attuale prima di questa PR: /

Comportamento desiderato dopo questa PR: /

--
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

@SilvioGregorini SilvioGregorini force-pushed the 11.0-account_balance_report branch 2 times, most recently from 202211a to a9d4c68 Compare July 6, 2019 16:27
@SilvioGregorini SilvioGregorini force-pushed the 11.0-account_balance_report branch 2 times, most recently from 97956dd to b9c9884 Compare August 20, 2019 13:29
@SilvioGregorini SilvioGregorini force-pushed the 11.0-account_balance_report branch 2 times, most recently from b5519af to 534e1fc Compare December 23, 2019 16:30
@SilvioGregorini SilvioGregorini mentioned this pull request Jan 17, 2020
Merged
marcelofrare
marcelofrare approved these changes Feb 28, 2020
View reviewed changes
Copy link

@marcelofrare marcelofrare left a comment

Choose a reason for hiding this comment

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

Funziona correttamente... la stiamo usando in produzione da diversi mesi!
Grazie @SilvioGregorini !

andreampiovesana
andreampiovesana approved these changes Feb 28, 2020
View reviewed changes
Copy link
Contributor

@andreampiovesana andreampiovesana left a comment

Choose a reason for hiding this comment

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

go

@marcelofrare
Copy link

visto che ci sono... prima che venga fatto il merge, vorrei verificare che la stampa esca correttamente in ITA. Vi faccio sapere...

@marcelofrare
Copy link

visto che ci sono... prima che venga fatto il merge, vorrei verificare che la stampa esca correttamente in ITA. Vi faccio sapere...

confermo che: HTML, PDF e XLSX escono tradotti correttamente.

@SilvioGregorini potresti verificare allinemento con i commit fatti sulla v.12 #1571?

@OpenCode
Copy link
Contributor

/ocabot merge

@OCA-git-bot
Copy link
Contributor

Hi @OpenCode. Your command failed:

Required option bumpversion_mode for command merge. Possible values : major, minor, patch, nobump.

Ocabot commands

  • ocabot merge major|minor|patch|nobump

More information

@OpenCode
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Oct 12, 2020
@OCA-git-bot
Merge PR #1282 into 11.0
7b8f780 
Signed-off-by OpenCode
@OCA-git-bot
Copy link
Contributor

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

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

@OpenCode
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 11.0-ocabot-merge-pr-1282-by-OpenCode-bump-nobump, awaiting test results.

@OCA-git-bot
Copy link
Contributor

This PR looks fantastic, let's merge it!
Prepared branch 11.0-ocabot-merge-pr-1282-by-OpenCode-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 16, 2020
@OCA-git-bot
Merge PR #1282 into 11.0
785bf30 
Signed-off-by OpenCode
@OCA-git-bot
Copy link
Contributor

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

@OpenCode
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

What a great day to merge this nice PR. Let's do it!
Prepared branch 11.0-ocabot-merge-pr-1282-by-OpenCode-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Oct 19, 2020
@OCA-git-bot
Merge PR #1282 into 11.0
549e0e4 
Signed-off-by OpenCode
@OCA-git-bot
Copy link
Contributor

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

@eLBati
Copy link
Member

eLBati commented Oct 22, 2020

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

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

OCA-git-bot added a commit that referenced this pull request Oct 22, 2020
@OCA-git-bot
Merge PR #1282 into 11.0
940633a 
Signed-off-by eLBati
@OCA-git-bot
Copy link
Contributor

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

@eLBati
Copy link
Member

eLBati commented Oct 24, 2020

Direi che bisogna risolvere #1921

@SilvioGregorini SilvioGregorini force-pushed the 11.0-account_balance_report branch 2 times, most recently from 1ba0cc7 to 88071d4 Compare November 10, 2020 09:23
@OpenCode
Copy link
Contributor

/ocabot merge nobump

@OCA-git-bot
Copy link
Contributor

Hey, thanks for contributing! Proceeding to merge this for you.
Prepared branch 11.0-ocabot-merge-pr-1282-by-OpenCode-bump-nobump, awaiting test results.

OCA-git-bot added a commit that referenced this pull request Nov 11, 2020
@OCA-git-bot
Merge PR #1282 into 11.0
5c1ef85 
Signed-off-by OpenCode
@OCA-git-bot
Copy link
Contributor

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

patrickt-oforce pushed a commit to patrickt-oforce/l10n-italy that referenced this pull request Mar 9, 2022
@SilvioGregorini
[UPD][l10n_it_account_balance_report] Aligned to OCA version (see PR O…
90cd37e 
…CA#1282)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andreampiovesana andreampiovesana andreampiovesana approved these changes

@marcelofrare marcelofrare marcelofrare approved these changes

Assignees
No one assigned
Labels
approved needs review ready to merge
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
6 participants
@SilvioGregorini @marcelofrare @OpenCode @OCA-git-bot @eLBati @andreampiovesana