Skip to content

14.0 fix l10n it fatturapa mig scripts#1

Open
SimoneVagile wants to merge 3 commits into14.0from
14.0-fix-l10n_it_fatturapa-mig-scripts
Open

14.0 fix l10n it fatturapa mig scripts#1
SimoneVagile wants to merge 3 commits into14.0from
14.0-fix-l10n_it_fatturapa-mig-scripts

Conversation

@SimoneVagile
Copy link
Copy Markdown
Owner

Descrizione del problema o della funzionalità:

Comportamento attuale prima di questa PR:

Comportamento desiderato dopo questa PR:

--
Confermo di aver firmato il CLA https://odoo-community.org/page/cla e di aver letto le linee guida su https://odoo-community.org/page/contributing

Copy link
Copy Markdown

@tafaRU tafaRU left a comment

Choose a reason for hiding this comment

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

Ho estrapolato i commit con:
git format-patch --keep-subject --stdout origin/14.0..l10n-italy/8.0 -- migrations | git am -3 --keep --ignore-whitespace droppando il commit Backporting module from v10 nel quale veniva rimosso lo script di migrazione.

Direi pertano che è 👍

Copy link
Copy Markdown

@tafaRU tafaRU left a comment

Choose a reason for hiding this comment

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

Vedo però che precommit fallisce. Questo errore in particolare sarebbe da sistemare. Che ne pensi?

@tafaRU
Copy link
Copy Markdown

tafaRU commented Jul 7, 2021

Inoltre vale quanto anche qui quanto ho scritto su fredzamoabg/partner-contact#3 (comment)

@SimoneVagile
Copy link
Copy Markdown
Owner Author

Vedo però che precommit fallisce. Questo errore in particolare sarebbe da sistemare. Che ne pensi?

Lasciare quella riga comporta qualcosa nell'esecuzione del codice? Perché se è solo per restyiling, come il resto degli errore di pre-commit, si potrebbe sorvolare (la wheel è già stata generata per test).

Inoltre vale quanto anche qui quanto ho scritto su fredzamoabg/partner-contact#3 (comment)

Si potrebbe usare il task https://www.agilebg.com/web#id=15366&view_type=form&model=project.task&action=303&active_id=774&menu_id=702? Così chi lo ha in carico rimette le versioni stabili dei moduli una volta per tutti (segniamo sul task i moduli che devono fare il cambio).

@tafaRU
Copy link
Copy Markdown

tafaRU commented Jul 7, 2021

Lasciare quella riga comporta qualcosa nell'esecuzione del codice?

Direi di no dal momento che quell'encoding è il default di Python 3.

si potrebbe sorvolare (la wheel è già stata generata per test).

Ok

Si potrebbe usare il task https://www.agilebg.com/web#id=15366&view_type=form&model=project.task&action=303&active_id=774&menu_id=702?

Come vi viene meglio 😃 La cosa importante è non dimenticarsi di ripristinare le versioni stabili!

@SimoneVagile
Copy link
Copy Markdown
Owner Author

@tafaRU FYI https://docs.google.com/document/d/1v4gMMOKSdTIfD_wrSgI7DW8zOx6eKNiiqI27OeSs6QU/edit#heading=h.qodag8g8j5k7

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants