Add pre-migrate script to drop column bvr_adherent_num #4
Conversation
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): |
There was a problem hiding this comment.
I campi res.bank.ccp e res.partner.bank.ccp non sono più presenti in 14.0 quindi forse sono stati spostati o rimossi, come mai non sono gestiti nella migrazione?
There was a problem hiding this comment.
Perché sono stati integrati nel core di Odoo: OCA@bebd92b
(vedi il link alla commit anche sul task)
There was a problem hiding this comment.
(vedi il link alla commit anche sul task)
Avevo guardato la descrizione dell'attività ma non c'erano riferimenti ai campi ccp.
I dati che abbiamo in quei campi in v10, come vengono portati nelle nuove colonne del core, se ci vengono portati?
There was a problem hiding this comment.
Ho aggiunto il campo allo script (è valorizzato su record che verranno poi eliminati).
There was a problem hiding this comment.
Come mai il campo res.bank.ccp non è gestito nella migrazione?
There was a problem hiding this comment.
Mi era sfuggito, grazie
There was a problem hiding this comment.
Se gli unici record di res.partner.bank che avevano i campi bvr_adherent_num e ccp valorizzati sono da eliminare, allora può andare bene eliminare le loro colonne; è un po' rischioso perché da oggi alla migrazione si potrebbero aggiungere record che valorizzano questi campi e ci interessa migrarli, ma per me è sufficiente che @stevech091 ne sia al corrente.
Questo non vuol dire che debba succedere lo stesso per le colonne di res.bank.
@stevech091 puoi confermare che non ci interessa portare questi dati in 14.0?
There was a problem hiding this comment.
Da come eravamo rimasti quando ci siamo sentiti, questi dati sono dati bancari che non gli servono e che, al massimo, può recuperare tramite fatture, estratti conto ecc...
Per ccp di res.bank abbiamo un solo record che ha quel campo valorizzato.
Comunque, come dici tu, aspettiamo la conferma definitiva di @stevech091
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): |
There was a problem hiding this comment.
(vedi il link alla commit anche sul task)
Avevo guardato la descrizione dell'attività ma non c'erano riferimenti ai campi ccp.
I dati che abbiamo in quei campi in v10, come vengono portati nelle nuove colonne del core, se ci vengono portati?
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): |
There was a problem hiding this comment.
Come mai il campo res.bank.ccp non è gestito nella migrazione?
SimoneVagile
force-pushed
the
14.0-mig-l10n_ch_base_bank-scripts
branch
from
c8d2d06
to
afa0f17
Compare
|
|
||
|
|
||
| @openupgrade.migrate() | ||
| def migrate(env, version): |
There was a problem hiding this comment.
Se gli unici record di res.partner.bank che avevano i campi bvr_adherent_num e ccp valorizzati sono da eliminare, allora può andare bene eliminare le loro colonne; è un po' rischioso perché da oggi alla migrazione si potrebbero aggiungere record che valorizzano questi campi e ci interessa migrarli, ma per me è sufficiente che @stevech091 ne sia al corrente.
Questo non vuol dire che debba succedere lo stesso per le colonne di res.bank.
@stevech091 puoi confermare che non ci interessa portare questi dati in 14.0?
https://www.agilebg.com/web#id=17484&view_type=form&model=project.task&action=306&menu_id=702