Pedro M. Baeza

Results 2106 comments of Pedro M. Baeza

I still think it doesn't bring enough value to keep it, at least on our migration path, which should be: consider this module as merged in `website_sale`, and convert the...

OK, apart from the `product.document` records, you have to create the corresponding `ir.attachment`, linked to the products. Remember that the content is not duplicated, as it's a reference to the...

Yes, that's it. I know you are seeing this as something tremendous, but the cases where the same file is shared between several products is not so frequent, and if...

Well, but it has been working till now, so it's your module that is not assigning that `request` object or doing something weird, don't you think?

/ocabot migration website_sale_charge_payment_fee And what about the reviews?

@rousseldenis requested changes, so he should now check and remove/approve the review

Well, let's try a last ping to @rousseldenis

#939 already migrates this module, but both have red CI