Iván Todorovich

Results 89 comments of Iván Todorovich

Thanks for the news! This is great, backbone was so outdated

I agree with @legalsylvain , specially because the module dependencies have changed from `point_of_sale` to `pos_epson_printer`. And that last one might not be needed and not wanted for most people

The branch is still messy unfortunately. Make sure you push with `--force` after rebasing. Ideally, for this PR, only 1 commit should be here: `[ADD] pos_mail_receipt`

I've never encountered this myself but I see something that bugs me here: https://github.com/OCA/pos/blob/df17842e7cdce02eae825c1133c905dc2ae884f5/.travis.yml#L16 I'm not sure that's the right way of adding `wkhtmltopdf` dependency, as Odoo requires very specific...

It may make sense for older odoo versions, but testing the pos is getting easier and easier. For `>= 15.0` is actually pretty straightforward IMO.

@Cedric-Pigeon could you remove these files, as they should've been moved instead: https://github.com/acsone/pos/tree/14.0-pos_reason-cpi/pos_cash_move_reason/demo

Indeed both modules are doing pretty much the same thing. I agree with @legalsylvain , it's better to migrate the existing module and improve it instead

Hi @hparfr , Indeed it's slightly different.. Coincidently enough I've just got a request for a similar feature, only we need to configure the customer_required per product (not per pricelist,...

Given that there may be more than 1 approach to require customers in the PoS, and that they're all valid and doesn't make sense to include them in the same...

> Hello, @ivantodorovich can you please check it again so we should merge it ASPS. Hi, the implementation changed a lot and it was significatly simplified. I don't use this...