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

[14.0][FIX] l10n_it_central_journal_reportlab: Support lines without account #4296

Closed
wants to merge 1 commit into from
Closed

[14.0][FIX] l10n_it_central_journal_reportlab: Support lines without account #4296

wants to merge 1 commit into from

Conversation

tafaRU
Copy link
Member

@tafaRU tafaRU commented Jul 25, 2024 •
edited
Loading

Corregge #4289 per 14.0.

@OCA-git-bot
Copy link
Contributor

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

@tafaRU tafaRU mentioned this pull request Jul 25, 2024
Closed
2 tasks
@francesco-ooops francesco-ooops linked an issue Jul 25, 2024 that may be closed by this pull request
Errore stampa libro giornale raggruppato per righe senza conto #4289
Closed
2 tasks
@tafaRU tafaRU marked this pull request as draft July 25, 2024 13:10
@tafaRU tafaRU force-pushed the 14.0-fix-l10n_it_central_journal_reportlab-no_account branch 2 times, most recently from 950e302 to 6775372 Compare July 25, 2024 13:23
MaurizioPellegrinet
MaurizioPellegrinet approved these changes Jul 25, 2024
View reviewed changes
Copy link

@MaurizioPellegrinet MaurizioPellegrinet left a comment

Choose a reason for hiding this comment

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

LGTM

andreampiovesana
andreampiovesana approved these changes Jul 26, 2024
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.

LGTM

@TheMule71 TheMule71 mentioned this pull request Oct 4, 2024
Open
32 tasks
@TheMule71
Copy link
Contributor

Vd anche #4400

@francesco-ooops
Copy link
Contributor

/ocabot rebase

@OCA-git-bot
Copy link
Contributor

Congratulations, PR rebased to 14.0.

@OCA-git-bot OCA-git-bot force-pushed the 14.0-fix-l10n_it_central_journal_reportlab-no_account branch from 6775372 to 74e1ea1 Compare October 23, 2024 14:02
@francesco-ooops
Copy link
Contributor

@tafaRU puoi verificare se non è già stata risolta in #3922 con la PR #3250 ?

@SirAionTech SirAionTech added the is porting This pull request is porting a change from another version label Nov 8, 2024
@SirAionTech
Copy link
Contributor

Chiudo perché il problema in 14.0 è stato risolto da #3250, vedi #4289 (comment).

@tafaRU tafaRU deleted the 14.0-fix-l10n_it_central_journal_reportlab-no_account branch November 22, 2024 08:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@MaurizioPellegrinet MaurizioPellegrinet MaurizioPellegrinet approved these changes

@andreampiovesana andreampiovesana andreampiovesana approved these changes

Assignees
No one assigned
Labels
approved is porting This pull request is porting a change from another version
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Errore stampa libro giornale raggruppato per righe senza conto
7 participants
@tafaRU @OCA-git-bot @TheMule71 @francesco-ooops @SirAionTech @andreampiovesana @MaurizioPellegrinet