Maksym Yankin

Results 79 comments of Maksym Yankin

`stock_move_location` is migrating here: - [ ] https://github.com/OCA/stock-logistics-warehouse/pull/2244

The tests are failing because of the issue I've mentioned in this PR (`delivery_postlogistics`) - [ ] https://github.com/OCA/delivery-carrier/pull/974

@StephaneMangin if the field `parcel_tracking = fields.Char()` is defined in postlogistics I need to inherit

**NOTE for reviewers** Tests are failing and BTW not even run, because of `Deprecation Warnings` coming from `delivery_postlogistics`. Locally tested and should be ok.

> Hi @yankinmax How are you ? You can pull this dependency to avoid depending from postlogistics #983 I'll manage a PR to remove this field from postlogistics. For now...

Sorry about it. It’s definitely accidentally. I’ll be more attentive. Sorry again пт, 13 черв. 2025 р. о 17:43 Camille Morand ***@***.***> пише: > *Camille0907* left a comment (OCA/delivery-carrier#984) >...

> @yankinmax @Camille0907 I just pushed 2 new commits on this PR Thanks @gurneyalex

@ivantodorovich I wasn't sure to understand the cases where we need to check the stage is changed, so I have simplified the code. Thanks for pointing to this, I'll update.

> @yankinmax can you take care of the quick fix? 🙏🏻 on it right now

@SilvioC2C PR is updated, thanks for the perfect suggestions.