l10n-italy
l10n-italy copied to clipboard
OCA
[14.0][imp] l10n_it_sdi_channel, move pec channel to l10n_it_fatturapa_pec
@OCA/local-italy-maintainers c'è modo di programmare/dare una scadenza temporale alle review degli improvements?
@OCA/local-italy-maintainers ci riprovo, opinioni/review su questa modifica?
Grazie della PR! Ho provato ad aggiornare un DB esistente con questa modifica e ottengo:
psycopg2.errors.UniqueViolation: duplicate key value violates unique constraint "ir_config_parameter_key_uniq" DETAIL: Key (key)=(sdi.pec.first.address) already exists.
Riesci a correggere?
Penso venga sollevato perché il parametro sdi.pec.first.address esiste già con XMLID
l10n_it_sdi_channel.sdi_pec_first_addressma questo codice cerca di creare un altro parametro di configurazione con lo stesso nome, che però ha XMLIDl10n_it_fatturapa_pec.sdi_pec_first_address; se è così credo si possa risolvere con uno script di migrazione.
Come hai un attimo puoi riprovare?
I commit sono tutti da mantenere o si possono schiacciare? Se possibile, sarebbe bene avere un commit per ogni modulo come consigliato nelle linee guida:
Try to split into different commits where impacted modules are different
(rif. https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message)
Puoi aggiornare anche i README (nota che usano https://github.com/OCA/maintainer-tools#readme-generator) dei moduli coinvolti? Al momento il README di l10n_it_sdi_channel spiega come configurare i server di posta per la PEC, ma almeno quella parte penso andrebbe spostata in l10n_it_fatturapa_pec.
Si, i commit li schiaccio come viene approvata la pr e pronta per il merge. Ora sistemo il resto che hai segnalato.
I commit sono tutti da mantenere o si possono schiacciare? Se possibile, sarebbe bene avere un commit per ogni modulo come consigliato nelle linee guida:
Try to split into different commits where impacted modules are different
(rif. https://github.com/OCA/odoo-community.org/blob/master/website/Contribution/CONTRIBUTING.rst#71commit-message)
Puoi aggiornare anche i README (nota che usano https://github.com/OCA/maintainer-tools#readme-generator) dei moduli coinvolti? Al momento il README di l10n_it_sdi_channel spiega come configurare i server di posta per la PEC, ma almeno quella parte penso andrebbe spostata in l10n_it_fatturapa_pec.
Dovrei aver sistemato tutto, incluso 1 commit per modulo. Puoi ri-verificare? Grazie
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
@TheMule71 considerato che la v12 avrebbe la necessità delle stesse modifiche estetiche / di traduzione, possiamo fare merge di questa pr ed aprirne un'altra con le modifiche richieste?
@TheMule71 considerato che la v12 avrebbe la necessità delle stesse modifiche estetiche / di traduzione, possiamo fare merge di questa pr ed aprirne un'altra con le modifiche richieste?
Si si. Non era bloccante.
@tafaRU per questa c'è ancora qualcosa da modificare a tua opinione?
@francesco-ooops rimane ancora irrisolto https://github.com/OCA/l10n-italy/pull/2708#discussion_r937836665. Grazie.
On my way to merge this fine PR! Prepared branch 14.0-ocabot-merge-pr-2708-by-tafaRU-bump-minor, awaiting test results.
This PR has the approved label and has been created more than 5 days ago. It should therefore be ready to merge by a maintainer (or a PSC member if the concerned addon has no declared maintainer). 🤖
Congratulations, your PR was merged at 789764ef31420ef9ef7f81ab045dd8d7fe27a3f7. Thanks a lot for contributing to OCA. ❤️