Pedro M. Baeza

Results 2722 comments of Pedro M. Baeza

About the formal aspect, the tag of the commit should be `[ADD]`, not `[NEW]`. About the module name and feature, `stock_picking_state_tracking` can be more appropriated if the implementation is with...

Well, I think that should be a pro-active action, checking a list of "completed orders" more than a reactive one, having a notification per order (even with sound), but anyone...

I see something weird in that flows, but I'm not blocking anyway. I don't see normal that someone has to stop their work to do "something" on that case. Automate...

Uhm, OK, that seems a real need, although missing the notification/sound seems a real possibility, so you should consider a fallback.

This is depending of another pull request that must be merged before as I see in the CI (but no reference in the initial comment)

This have to go to OCA/mail instead.

You have to move this to OCA/mail

The initial balance in currency is incorrect: ![image](https://github.com/user-attachments/assets/890d92f2-74e4-43c5-89fc-65f6776fd42d)