Pedro M. Baeza

I'm afraid not. That's a hook, not a migration script. A sample (not with the same change though): https://github.com/OCA/contract/tree/16.0/contract/migrations

Please squash commits into one

Rescuing this: /ocabot merge major

@AnizR can you please check the problem with your module `account_payment_method_fs_storage`? https://github.com/OCA/bank-payment/actions/runs/13748693138/job/38446868447?pr=1360#step:8:438 `ValueError: External ID not found in the system: fs_storage.default_fs_storage` I think it should be a change in the...

It's better to not use demo data in tests. Demo data may have also any problem being loaded.

/ocabot merge major

Can you please fw-port it to 17? In this case, the script should check if the column is not JSONB for calling the method.

Usually `res` is not used in module names. We use `base_something` or directly the name. The module may be called simply `property` if you want.

/ocabot migration product_form_account_move_line_link /ocabot merge nobump

Please provide more info about this and greenify CIs.