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

[14.0][MIG] Migration to 14.0 l10n_it_fatturapa_out_fd #4386

Open
wants to merge 6 commits into
base: 14.0
Choose a base branch
from

Conversation

sergiocorato
Copy link
Contributor

No description provided.

@sergiocorato sergiocorato force-pushed the 14.0-mig-l10n_it_fatturapa_out_fd branch 4 times, most recently from 499801c to e34dc54 Compare September 27, 2024 13:01
@sergiocorato sergiocorato marked this pull request as draft September 30, 2024 07:03
@sergiocorato sergiocorato force-pushed the 14.0-mig-l10n_it_fatturapa_out_fd branch from e34dc54 to bf0f225 Compare September 30, 2024 08:36
@sergiocorato sergiocorato marked this pull request as ready for review September 30, 2024 08:36
@sergiocorato
Copy link
Contributor Author

@TheMule71 pronto per la review!

@sergiocorato
Copy link
Contributor Author

Ho notato che il numero della fattura, anche se assegnato, viene impostato in automatico ad un numero diverso (con la sequenza 'per mese' di default), cosa che non si nota nei vari test dove non si fa un controllo completo dell'xml.
Questa assegnazione in pratica è ignorata:

Copy link
Contributor

@SirAionTech SirAionTech left a comment

Choose a reason for hiding this comment

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

Grazie della PR!

Comment on lines 1 to 4
<ns1:FatturaElettronica xmlns:ns1="http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2" versione="FPR12">
<ns1:FatturaElettronica
xmlns:ns1="http://ivaservizi.agenziaentrate.gov.it/docs/xsd/fatture/v1.2"
versione="FPR12"
>
Copy link
Contributor

Choose a reason for hiding this comment

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

I cambiamenti fatti in automatico da pre-commit andrebbero lasciati in un commit a sé precedente la migrazione, come indicato in https://github.com/OCA/maintainer-tools/wiki/Migration-to-version-14.0:

$ git clone https://github.com/OCA/$REPO -b 14.0
$ git checkout -b 14.0-mig-$MODULE origin/14.0
$ git format-patch --keep-subject --stdout origin/14.0..origin/13.0 -- $MODULE | git am -3 --keep
$ pre-commit run -a  # to run black, isort and prettier (ignore pylint errors at this stage)
$ git add -A
$ git commit -m "[IMP] $MODULE: black, isort, prettier"  --no-verify  # it is important to do all formatting in one commit the first time

riesci a correggere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dovrei aver fatto

Copy link
Contributor

Choose a reason for hiding this comment

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

Puoi schiacciare i commit fatti dai bot come indicato in https://github.com/OCA/maintainer-tools/wiki/Merge-commits-in-pull-requests?
A occhio della storia di 12.0 dovrebbe rimanere solo il commit ADD e un commit di traduzione, invece dei 5 che ci sono adesso
image

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fatto

@api.model
def getTemplateValues(self, template_values):
template_values = super().getTemplateValues(template_values)
template_values.update({"get_importo": self.get_importo})
Copy link
Contributor

Choose a reason for hiding this comment

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

Così facendo sovrascrivi la funzione get_importoche c'è già in template_values:

"get_importo": get_importo,
quindi va a farsi benedire 👼 qualsiasi modifica che altri moduli potrebbero voler fare a questa funzione, così come eventuali modifiche della funzione originale stessa.
Potresti invece mantenere l'ereditarietà? Ad esempio puoi vedere come è stato fatto per get_importo_totale:
def get_importo_totale(invoice):
# wrapper to a method in wizard (for better overriding)
wiz = self.env["wizard.export.fatturapa"]
return wiz.getImportoTotale(invoice)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

dovrei aver fatto

invoice_form = Form(
self.env["account.move"].with_context(default_move_type="out_invoice")
)
invoice_form.name = "INV/2020/01/0001"
Copy link
Contributor

Choose a reason for hiding this comment

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

Ho notato che il numero della fattura, anche se assegnato, viene impostato in automatico ad un numero diverso (con la sequenza 'per mese' di default), cosa che non si nota nei vari test dove non si fa un controllo completo dell'xml. Questa assegnazione in pratica è ignorata:

Se come scrivi viene ignorata, si può rimuovere?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

fatto

@sergiocorato sergiocorato force-pushed the 14.0-mig-l10n_it_fatturapa_out_fd branch 2 times, most recently from 9fafba4 to 420f135 Compare October 24, 2024 15:04
@sergiocorato sergiocorato force-pushed the 14.0-mig-l10n_it_fatturapa_out_fd branch from dbbf7a4 to 2418018 Compare October 24, 2024 15:18
@sergiocorato sergiocorato force-pushed the 14.0-mig-l10n_it_fatturapa_out_fd branch from e5e12ea to 100ea0f Compare October 24, 2024 16:24
@sergiocorato
Copy link
Contributor Author

@SirAionTech dovrei aver fatto tutto

@SirAionTech SirAionTech self-requested a review October 25, 2024 07:04
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.

Integrazione della fattura elettronica con lo sconto fisso
5 participants