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

[16.0][ADD]l10n_it_intrastat_delivery_note: new module adds missing fields to the delivery note report #4483

Open
wants to merge 1 commit into
base: 16.0
Choose a base branch
from
Open

[16.0][ADD]l10n_it_intrastat_delivery_note: new module adds missing fields to the delivery note report #4483

wants to merge 1 commit into from

Conversation

PicchiSeba
Copy link
Contributor

fw-port of #3530

andreampiovesana
andreampiovesana suggested changes Dec 4, 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.

cosa è il file pandoc-3.5-1-amd64.deb?

@PicchiSeba
Copy link
Contributor Author

@andreampiovesana
Non ne ho idea ma è sicuramente fuori posto. Lo rimuovo. Grazie per la segnalazione

@PicchiSeba PicchiSeba force-pushed the 16.0-add-intractat_delivery_note branch from 87413c5 to 2d76179 Compare December 5, 2024 08:10
@PicchiSeba PicchiSeba marked this pull request as draft December 5, 2024 08:10
@PicchiSeba
Copy link
Contributor Author

Gli stessi test sembrano rotti anche in altre PR #4474

@SirAionTech
Copy link
Contributor

Gli stessi test sembrano rotti anche in altre PR #4474

Al momento i test di 16.0 stanno fallendo per #4461.

In questa PR però c'è un altro errore

Element '<xpath expr="//group/group/field[@name='payment_reference']">' cannot be located in parent view

(da https://github.com/OCA/l10n-italy/actions/runs/12175686656/job/33959904382?pr=4483#step:8:178)

@PicchiSeba PicchiSeba force-pushed the 16.0-add-intractat_delivery_note branch 2 times, most recently from 2d76179 to dd2db0e Compare December 6, 2024 09:43
@PicchiSeba
Copy link
Contributor Author

Buongiorno @SirAionTech , ho cercato di riprodurre il problema in locale installando questi moduli ma non mi esce l'errore. Tra l'altro in questo modulo non sono andato a modificare nessuna view di account, solamente dei report.

Sto sbagliando qualcosa?

@micheledic
Copy link
Contributor

@PicchiSeba fai rebase e l'errore va via

@PicchiSeba PicchiSeba force-pushed the 16.0-add-intractat_delivery_note branch from dd2db0e to c0568f1 Compare December 6, 2024 15:22
@PicchiSeba PicchiSeba changed the title [16.0][ADD]l10n_it_intrastat_delivery_note: new module that adds missing fields to the delivery note report [16.0][ADD]l10n_it_intrastat_delivery_note: new module adds missing fields to the delivery note report Dec 9, 2024
@PicchiSeba PicchiSeba force-pushed the 16.0-add-intractat_delivery_note branch 2 times, most recently from ca08dd8 to b69db31 Compare December 9, 2024 16:10
@PicchiSeba PicchiSeba force-pushed the 16.0-add-intractat_delivery_note branch from b69db31 to 96ce878 Compare December 10, 2024 07:47
@PicchiSeba PicchiSeba marked this pull request as ready for review December 10, 2024 07:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@andreampiovesana andreampiovesana andreampiovesana requested changes

Assignees
No one assigned
Labels
16.0 new module
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@PicchiSeba @SirAionTech @micheledic @andreampiovesana @francesco-ooops