Open techgrrow opened 1 month ago
perfavore puoi sistemare il nome dei commit? Mi sembra inoltre che ci sono già delle pr che sistemano questo puoi fare review di quelle cosi le mergiamo?
@matteoopenf riesci a ritrovare queste PR?
@matteoopenf riesci a ritrovare queste PR?
Ciao, io ricordo di averle viste le pr relative a questo modulo con problemi simili. Ma non le trovo, mi scuso, questo punto serve aprire una isssue e sistemare il commit
Ciao @matteoopenf potresti spiegarmi cosa dovrei fare per sistemare il commit/nome del commit? E la issue? Non sono pratico con i contributi OCA. Grazie
Ciao @matteoopenf potresti spiegarmi cosa dovrei fare per sistemare il commit/nome del commit? E la issue? Non sono pratico con i contributi OCA. Grazie
Buongiorno, da qui https://github.com/OCA/l10n-italy/issues Cliccando su New Issue puoi seguire gli step per creare una issue e indichi poi le versioni affette dal problema.
https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#commit-message Qui trovi delle info sui commit. @francesco-ooops ops mi sembra abbia un link più stringato delle informazioni.
Grazie @matteoopenf ho modificato il commit e creato una issue qui https://github.com/OCA/l10n-italy/issues/4424, per piacere fammi sapere se vanno bene.
Grazie @matteoopenf ho modificato il commit e creato una issue qui #4424, per piacere fammi sapere se vanno bene.
ti ho commentato la issue la pr o è un fix è un add secondo me solitamente non è mai entrambi se fai sia dei fix che degli add fai dei commit distinti
Grazie @matteoopenf ho editato sia la issue sia la PR, come scritto nella issue le modifiche inizialmente segnate come "add" sono dei fix, nel senso che allineano il comportamento del report raggruppato a quello completo: in particolare gli ultimi due punti dove si popola la colonna "name" che altrimenti rimarrebbe vuota e il poplarla con il nome del partner nei casi di conti di debito o credito, come nella stampa non raggruppata. Disponibile comunque a spacchettare se serve
@matteoopenf ho rieditato la PR rimuovendo le logiche per il campo "name" (al suo posto ho messo un metodo vuodo per computarle a piacere, _compute_aml_grouped_name()), così la PR dovrebbe apportare solo i fix della issue, lasciando il discorso della colonna name da gestire in seguito. Può andare?
@francesco-ooops avevo aggiunto in precendenza la issue, puoi togliere la label "missing issue"? Grazie
Aggiunto un parametro al metodo vuoto _compute_aml_grouped_name(line) così che sia più facile computare il campo "Name" con le le logiche desiderate
/ocabot rebase
@francesco-ooops The rebase process failed, because command git rebase origin/16.0
failed with output:
Rebasing (1/1)
Auto-merging l10n_it_central_journal_reportlab/wizard/print_giornale.py
CONFLICT (content): Merge conflict in l10n_it_central_journal_reportlab/wizard/print_giornale.py
error: could not apply 463e33ab3... [FIX]l10n_it_central_journal_reportlab: formatting and partner logic
hint: Resolve all conflicts manually, mark them as resolved with
hint: "git add/rm <conflicted_files>", then run "git rebase --continue".
hint: You can instead skip this commit: run "git rebase --skip".
hint: To abort and get back to the state before "git rebase", run "git rebase --abort".
Could not apply 463e33ab3... [FIX]l10n_it_central_journal_reportlab: formatting and partner logic
@techgrrow puoi fare rebase?
@francesco-ooops certo, fatto
Ciao @francesco-ooops devo fare altro perchè la PR venga mergiata? Grazie
@techgrrow servono due review, di cui almeno una di codice (non fa male averne almeno una funzionale). Non ho esperienza in questo ambito e non posso aiutarti, magari @matteoopenf o @OCA/local-italy-maintainers possono fare review?
Ok @matteoopenf grazie mille, aspetto quindi le review. @matteoopenf @OCA/local-italy-maintainers sono a disposizione se serve un aiuto o chiarimenti da parte mia.
-fixed grouped formatting (moved values under correct columns, shifted to the right) -fixed/moved content of column "name" under "ref" (as it contains the account_move ref field values)