Skip to content

[FIX] l10n_it_central_journal_reportlab: exclude notes and sections from report#4905

Open
micheledic wants to merge 1 commit intoOCA:18.0from
stesi:18.0_fix_central_journal_notes_names
Open

[FIX] l10n_it_central_journal_reportlab: exclude notes and sections from report#4905
micheledic wants to merge 1 commit intoOCA:18.0from
stesi:18.0_fix_central_journal_notes_names

Conversation

@micheledic
Copy link
Copy Markdown
Contributor

@micheledic micheledic commented Sep 11, 2025 •
edited
Loading

This PR fixes exclude the lines of notes and section from the export

@OCA-git-bot
Copy link
Copy Markdown
Contributor

OCA-git-bot commented Sep 11, 2025

Hi @Borruso, @MarcoCalcagni,
some modules you are maintaining are being modified, check this out!

@francesco-ooops francesco-ooops added missing issue PRs must be linked to an issue. 18.0 labels Sep 11, 2025
@Borruso
Copy link
Copy Markdown
Contributor

Borruso commented Sep 11, 2025

per l'errore su partner c'è #4863 aperta
puoi aggingere solo la fix dell'esclusione di note e sezioni in questa che hai aperto

@micheledic micheledic force-pushed the 18.0_fix_central_journal_notes_names branch from c2d7dd6 to e1f96b8 Compare September 11, 2025 10:42
@micheledic
Copy link
Copy Markdown
Contributor Author

micheledic commented Sep 11, 2025

per l'errore su partner c'è #4863 aperta puoi aggingere solo la fix dell'esclusione di note e sezioni in questa che hai aperto

Fatto,
comunque parzialmente il mio commit risolve anche il problema dell'errore su account_name vuoto, sarebbero le righe note e sezione che vengono prese dalla query ma che non servono nel libro giornale

@Borruso
Copy link
Copy Markdown
Contributor

Borruso commented Sep 11, 2025

per l'errore su partner c'è #4863 aperta puoi aggingere solo la fix dell'esclusione di note e sezioni in questa che hai aperto

Fatto, comunque parzialmente il mio commit risolve anche il problema dell'errore su account_name vuoto, sarebbero le righe note e sezione che vengono prese dalla query ma che non servono nel libro giornale

perfetto se puoi modifica il messaggio del commit

@micheledic
[FIX] l10n_it_central_journal_reportlab: exclude notes and sections f…
78aebf9 
…rom report

This PR fixes exclude the lines of notes and section from the export
@micheledic micheledic force-pushed the 18.0_fix_central_journal_notes_names branch from e1f96b8 to 78aebf9 Compare September 12, 2025 06:49
@micheledic micheledic changed the title [FIX] l10n_it_central_journal_reportlab: exclude notes and sections from report and avoid error if partner or move is false [FIX] l10n_it_central_journal_reportlab: exclude notes and sections from report Sep 12, 2025
@micheledic
Copy link
Copy Markdown
Contributor Author

micheledic commented Sep 12, 2025

per l'errore su partner c'è #4863 aperta puoi aggingere solo la fix dell'esclusione di note e sezioni in questa che hai aperto

Fatto, comunque parzialmente il mio commit risolve anche il problema dell'errore su account_name vuoto, sarebbero le righe note e sezione che vengono prese dalla query ma che non servono nel libro giornale

perfetto se puoi modifica il messaggio del commit

fatto

Borruso
Borruso approved these changes Jan 27, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@Borruso Borruso left a comment

Choose a reason for hiding this comment

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

LGTM

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

1 more reviewer

@Borruso Borruso Borruso approved these changes

Reviewers whose approvals may not affect merge requirements

Assignees

No one assigned

Labels

18.0 missing issue PRs must be linked to an issue.

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

4 participants

@micheledic @OCA-git-bot @Borruso @francesco-ooops